8000 stop pretending to support gzip *request* transfer-encoding by pajod · Pull Request #3368 · benoitc/gunicorn · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

stop pretending to support gzip *request* transfer-encoding #3368

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pajod
Copy link
Contributor
@pajod pajod commented Mar 20, 2025

This undoes (or restarts the conversation about) the part of #3260 that I do not understand.

If Gunicorn does not fully handle a hop-by-hop header, and leaves the WSGI application in no place to figure out what is left to do, it should not not partially handle it.

https://peps.python.org/pep-3333/#other-http-features

WSGI servers must handle any supported inbound “hop-by-hop” headers on their own, such as by decoding any inbound Transfer-Encoding, including chunked encoding if applicable.

https://datatracker.ietf.org/doc/html/rfc9110#section-7.6.1

intermediaries SHOULD remove or replace fields that are known to require removal before forwarding

* https://peps.python.org/pep-3333/#other-http-features

"WSGI servers must handle any supported inbound “hop-by-hop”
headers on their own, such as by decoding any inbound
Transfer-Encoding, including chunked encoding if applicable."

* https://datatracker.ietf.org/doc/html/rfc9110#section-7.6.1

"intermediaries SHOULD remove or replace fields that are
known to require removal before forwarding"

* partial revert: 555d2fa
@pajod pajod moved this to Awaiting: design decision in @pajod Gunicorn issue sorting table Mar 21, 2025
@pajod pajod changed the title stop pretending to support gzip stop pretending to support gzip *request* transfer-encoding Mar 21, 2025
@pajod pajod mentioned this pull request Apr 8, 2025
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0