Opened 13 years ago

Closed 9 years ago

#14968 closed Bug (needsinfo)

handle BaseExceptions with middleware, or at least KeyboardInterrupt

Reported by: Alexander Schepanovski Owned by: Alexander Schepanovski
Component: Core (Other) Version: 1.2
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

I was trying to write a middleware which shows a 503 page on KeyboardInterrupt (which happens on django restart) and run into this.
If it is not handled some unlucky users get 500 error page and search bots go away for a while.

Attachments (1)

catch_base_exception.diff (643 bytes ) - added by Alexander Schepanovski 13 years ago.

Download all attachments as: .zip

Change History (6)

by Alexander Schepanovski, 13 years ago

Attachment: catch_base_exception.diff added

comment:1 by Russell Keith-Magee, 13 years ago

Patch needs improvement: set
Triage Stage: UnreviewedAccepted

BaseException was introduced in Python 2.5, and Django needs to support Python 2.4 (at least for now).

Patch needs to be updated to support the exception class heirarchy in Python 2.4, as well as Python 2.5+

comment:2 by anonymous, 13 years ago

Severity: Normal
Type: Bug

comment:3 by Aymeric Augustin, 12 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:4 by Aymeric Augustin, 12 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:5 by Tim Graham, 9 years ago

Resolution: needsinfo
Status: newclosed

I don't understand the purpose of this change and there's been no activity in 5 years. Feel free to reopen with an explanation and an updated patch.

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