Opened 6 years ago

Closed 6 years ago

Last modified 2 years ago

#24733 closed New feature (fixed)

Pass exceptions to error handlers

Reported by: Claude Paroz Owned by: nobody
Component: HTTP handling Version: master
Severity: Normal Keywords:
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

As discussed in this thread: ​https://groups.google.com/forum/#!topic/django-developers/kcFqjmnlUTQ

Error views should receive the exception that triggered the view. This is especially useful to show custom messages for permission denied errors (403).

Change History (8)

comment:1 Changed 6 years ago by Claude Paroz

Has patch: set

PR: ​https://github.com/django/django/pull/4590

comment:2 Changed 6 years ago by Tim Graham

Triage Stage: Unreviewed → Accepted

comment:3 Changed 6 years ago by Claude Paroz <claude@…>

In 69125ce9:

Extended variable name in handlers/base.py

Refs #24733.

comment:4 Changed 6 years ago by Tim Graham

Triage Stage: Accepted → Ready for checkin

comment:5 Changed 6 years ago by Claude Paroz <claude@…>

Resolution: → fixed
Status: new → closed

In 70779d9:

Fixed #24733 -- Passed the triggering exception to 40x error handlers

Thanks Tim Graham for the review.

comment:6 Changed 4 years ago by Tim Graham <timograham@…>

In 3cee9ed:

Refs #24733 -- Removed support for error views without the exception parameter.

Per deprecation timeline.

comment:7 Changed 2 years ago by Tim Graham <timograham@…>

In 49f97b64:

Refs #24733 -- Documented arguments for custom error views.

comment:8 Changed 2 years ago by Tim Graham <timograham@…>

In 0d8f26f3:

[2.1.x] Refs #24733 -- Documented arguments for custom error views.

Backport of 49f97b645fb1716cf69177558c41cf650a701eed from master

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