Changes between Initial Version and Version 1 of Ticket #28513


Ignore:
Timestamp:
08/21/2017 06:49:36 AM (4 years ago)
Author:
Tim Graham
Comment:

It seems reasonable to support that.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #28513

    • Property Severity changed from Normal to Release blocker
    • Property Component changed from Uncategorized to contrib.auth
    • Property Summary changed from django 1.11 doesn't support POST for logout view to LogoutView doesn't support POST, unlike the function-based logout() view
    • Property Type changed from Uncategorized to Bug
    • Property Triage Stage changed from Unreviewed to Accepted
  • Ticket #28513 – Description

    initial v1  
    1 Hello,
     1The function-based `logout()` view allows using a POST request.
    22
    3 On Django versions < 1.11, the logout view allowed us to use a POST request.
    4 
    5 Because no ''post'' method is defined on the new ''LogoutView'' introduced in https://github.com/django/django/commit/78963495d0caadb77eb97ccf319ef0ba3b204fb5, a POST request will raise 405 Method not allowed error. I don't think POST was officially supported (don't see it in the docs) but there's code in the view that checks for the redirect field in the POST data.
     3Because the new `LogoutView` introduced in 78963495d0caadb77eb97ccf319ef0ba3b204fb5 doesn't define a `post()` method, a POST request returns "405 Method not allowed". I don't think POST was officially supported (don't see it in the docs) but there's code in the view that checks for the redirect field in the POST data.
Back to Top