Re: [Django] #36002: RemoteUserMiddleware/PersistentRemoteUserMiddleware docstrings and code comments refer to "headers" rather than request.META keys (was: Errors in RemoteUserMiddleware/PersistentRemoteUserMiddleware docstrings and code comments)

3 views
Skip to first unread message

Django

unread,
Dec 12, 2024, 8:53:58 AM12/12/24
to django-...@googlegroups.com
#36002: RemoteUserMiddleware/PersistentRemoteUserMiddleware docstrings and code
comments refer to "headers" rather than request.META keys
-------------------------------------+-------------------------------------
Reporter: Anders Einar Hilden | Owner: Anders
Type: | Einar Hilden
Cleanup/optimization | Status: new
Component: contrib.auth | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
RemoteUserMiddleware, |
PersistentRemoteUserMiddleware |
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Sarah Boyce):

* needs_better_patch: 0 => 1
* resolution: invalid =>
* stage: Unreviewed => Accepted
* status: closed => new
* summary:
Errors in RemoteUserMiddleware/PersistentRemoteUserMiddleware
docstrings and code comments
=>
RemoteUserMiddleware/PersistentRemoteUserMiddleware docstrings and
code comments refer to "headers" rather than request.META keys

Comment:

Accepting after further investigation
Also looks like some clarification in the
[https://docs.djangoproject.com/en/5.1/ref/request-
response/#django.http.HttpRequest.META HttpRequest.META] docs would be
beneficial to clarify that these are not exclusively headers and contain
things like [https://datatracker.ietf.org/doc/html/rfc3875#section-4.1
request Meta variables] but that discussion should perhaps happen on the
forum to iron things out
--
Ticket URL: <https://code.djangoproject.com/ticket/36002#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Reply all
Reply to author
Forward
0 new messages