Opened 8 months ago

Last modified 8 months ago

#35354 closed Cleanup/optimization

Simplify ASGIRequest path handling. — at Initial Version

Reported by: Carlton Gibson Owned by: nobody
Component: HTTP handling Version: 5.0
Severity: Normal Keywords: asgi
Cc: Andrew Godwin 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

Following the ASGI HTTP Connection Scope docs[0], the provided path is already the correct value that Django requires.

In combination with root_path, from which script_name is derived, the path_info variable can is set.
It's then redundant to re-calculate path from script_name and path_info.

See also, a clarifying discussion on the ASGIref repo[1].

[0]: https://asgi.readthedocs.io/en/latest/specs/www.html#http-connection-scope
[1]: https://github.com/django/asgiref/issues/424

Change History (0)

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