Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

management: Don't prepend "HTTP_" to certian headers in parse_headers. #12536

Closed
wants to merge 1 commit into from
Closed

management: Don't prepend "HTTP_" to certian headers in parse_headers. #12536

wants to merge 1 commit into from

Conversation

Hypro999
Copy link
Member

@Hypro999 Hypro999 commented Jun 8, 2019

Django does not prepend "HTTP_" to "Content-Type" or "Content-Length" headers. So parse_headers should not do that either This was just a small bug I noticed while working on something else.

Django does not prepend "HTTP_" to "Content-Type" or "Content-Length"
headers. So parse_headers should not do that either.
@timabbott
Copy link
Sponsor Member

Nice, merged as 1a33d73, after adding a comment linking to the right part of the Django docs for understanding that this code is correct.

Thanks @Hypro999!

@timabbott timabbott closed this Jun 14, 2019
@Hypro999 Hypro999 deleted the fix-parse-headers-issue branch June 20, 2019 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants