Opened 18 years ago

Closed 18 years ago

#2405 closed defect (fixed)

[patch] django.contrib.auth.views.logout causes recursion error

Reported by: anonymous Owned by: Adrian Holovaty
Component: Contrib apps Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

This is related to #2368 because that patch now catches the KeyError it is not passed to the view that was depending on it. So it looks like the view no longer needs the try catch.

The current state of Django is that when I try to logout I get recursive calls to logout.

[23/Jul/2006 13:40:11] "GET /accounts/logout/ HTTP/1.1" 302 0

[23/Jul/2006 13:40:11] "GET /accounts/logout/ HTTP/1.1" 302 0

[23/Jul/2006 13:40:11] "GET /accounts/logout/ HTTP/1.1" 302 0

[23/Jul/2006 13:40:11] "GET /accounts/logout/ HTTP/1.1" 302 0

[23/Jul/2006 13:40:11] "GET /accounts/logout/ HTTP/1.1" 302 0

Attachments (1)

logout.diff​ (738 bytes ) - added by jay@… 18 years ago.

Download all attachments as: .zip

Change History (3)

by jay@…, 18 years ago

Attachment: logout.diff​ added

comment:1 by jay@…, 18 years ago

Oops, this was submited by me, not anonymous.

comment:2 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: new → closed

(In [3431]) Fixed #2405 -- Fixed recursion error in auth.views.logout(), caused by [3402]. Thanks, Jay Skabber

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