search

jjheffernan / Django-Auto-Forum

Django Based Forum with content hosting to cloud, focused around Auto Enthusiasts
1 stars 0 forks source link

Login Redirect Issues #2

Closed jjheffernan closed 2 years ago

jjheffernan commented 2 years ago

The URL pathing between the different login buttons does not correctly interface, and requires fixing. sometimes /dashboard/ is injected into the URL pattern when trying to log in

jjheffernan commented 2 years ago

See the related comment issue. re-did registration using CBVs to include some class based views. Need to still expand and fully migrate. This may require nuking the db except for the superuser already registered within Django. Will update

jjheffernan commented 2 years ago

re-direct now goes as follows: login page -> success_url, request reload for invalid form with raised exception, OR direct to home using aforementioned global redirect within forum_base/settings.py

jjheffernan commented 2 years ago

Fixed basic login redirect issues. the error came from a mis-labelled route that would redirect "profile/accounts/login/dashboard" instead of "profile/dashboard". The bug was stuck in between 'user_profile/urls.py' and 'forum_base/settings.py'. It has been fixed in the current forum_dev push.

jjheffernan commented 2 years ago

Login OAuth still needs to be built out. The reset-password link still defaults to the Django admin site. This could theoretically cause security problems. Feature will be revisited, but issue is closed.