Opened 7 months ago

Closed 7 months ago

Last modified 7 months ago

#32754 closed Bug (fixed)

catch_all_view() does not support FORCE_SCRIPT_NAME.

Reported by: SlavaSkvortsov Owned by: SlavaSkvortsov
Component: contrib.admin Version: 3.2
Severity: Release blocker Keywords: FORCE_SCRIPT_NAME catch_all_view final_catch_all_view
Cc: Jon Dufresne 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 (last modified by SlavaSkvortsov)

catch_all_view returns redirect to '%s/' % request.path_info (script name cut off there) instead of '%s/' % request.path (with the script name)

Patch - https://github.com/django/django/pull/14404

Change History (6)

comment:1 Changed 7 months ago by Mariusz Felisiak

Cc: Jon Dufresne added
Owner: changed from nobody to SlavaSkvortsov
Severity: NormalRelease blocker
Status: newassigned
Summary: catch_all_view from admin does not support FORCE_SCRIPT_NAMEcatch_all_view() does not support FORCE_SCRIPT_NAME.
Triage Stage: UnreviewedAccepted

Thanks for the report, good catch! Bug in ba31b0103442ac891fb3cb98f316781254e366c3.

comment:2 Changed 7 months ago by SlavaSkvortsov

Description: modified (diff)

comment:3 Changed 7 months ago by SlavaSkvortsov

Thank you for a quick answer!

I wrote a patch here

comment:4 Changed 7 months ago by Mariusz Felisiak

Triage Stage: AcceptedReady for checkin

comment:5 Changed 7 months ago by Mariusz Felisiak <felisiak.mariusz@…>

Resolution: fixed
Status: assignedclosed

In f7691d4:

Fixed #32754 -- Made AdminSite.catch_all_view() respect SCRIPT_NAME.

Regression in ba31b0103442ac891fb3cb98f316781254e366c3.

comment:6 Changed 7 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In ce78bc98:

[3.2.x] Fixed #32754 -- Made AdminSite.catch_all_view() respect SCRIPT_NAME.

Regression in ba31b0103442ac891fb3cb98f316781254e366c3.

Backport of f7691d4812c578e696635718e67639d2e08eac40 from main

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