Opened 7 months ago
Last modified 7 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
Note:
See TracTickets
for help on using tickets.