Opened 6 years ago

Closed 6 years ago

#29252 closed Bug (duplicate)

Add CSRF protection to LogoutView

Reported by: Filip Dimitrovski Owned by: nobody
Component: contrib.auth Version: dev
Severity: Normal Keywords: denial of service, csrf
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Filip Dimitrovski)

The current implementation in django.contrib.auth.views.LogoutView allows both GET and POST requests without any CSRF protection. A simple 

   <img src="http://djangoapp/logout" />

on an exploit page could log out the user. While this is a low security risk, it's still a DoS issue and could prevent the user from using the app.

Instead of fixing the view, it maybe makes sense to just change the ​docs to warn the programmer of such a problem and suggest overriding LogoutView and changing dispatch().

Change History (3)

comment:1 by Filip Dimitrovski, 6 years ago

Easy pickings: set

comment:2 by Filip Dimitrovski, 6 years ago

Description: modified (diff)

comment:3 by Tim Graham, 6 years ago

Resolution: → duplicate
Status: new → closed

Duplicate of #15619.

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