Opened 9 years ago

Closed 9 years ago

Last modified 5 years ago

#6340 closed Uncategorized (wontfix)

auth.logout view doesn't use LOGOUT_URL in settings.py

Reported by: Alper KANAT <alperkanat@…> Owned by: David Tulig
Component: Generic views Version: master
Severity: Normal Keywords: auth, view, logout
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

title says it all :)

Attachments (1)

logout_url.diff (2.3 KB) - added by David Tulig 9 years ago.
Adds LOGOUT_URL support to the logout view.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 9 years ago by David Tulig

Has patch: set
Needs documentation: unset
Needs tests: unset
Owner: changed from nobody to David Tulig
Patch needs improvement: unset
Status: newassigned

Changed 9 years ago by David Tulig

Attachment: logout_url.diff added

Adds LOGOUT_URL support to the logout view.

comment:2 Changed 9 years ago by Simon Greenhill <dev@…>

Triage Stage: UnreviewedReady for checkin

comment:3 Changed 9 years ago by Malcolm Tredinnick

Settings should never be used at import time, which means don't use them as default arguments for functions. Otherwise you cannot import those modules without having already configured settings, which is inconvenient.

comment:4 Changed 9 years ago by Malcolm Tredinnick

Resolution: wontfix
Status: assignedclosed

Looking at this more closely, I think it's a wontfix. The logout() method, by default, displays a particular page (it doesn't do a redirect). If you want it to do a redirect, you can pass in the URL to redirect to, so the functionality is already possible. This patch would just change the default to be a redirect, which isn't necessarily a clear win, so would be a backwards incompatible change that would be more annoying than useful. Since either behaviour is possible at the moment, let's leave it alone.

comment:5 Changed 5 years ago by dana@…

Easy pickings: unset
Severity: Normal
Type: Uncategorized
UI/UX: unset

Maybe a note about this in the documentation would be appropriate as if there is a LOGOUT_URL setting but it doesn't do anything could be bit confusing for new users...

Note: See TracTickets for help on using tickets.
Back to Top