benoitc / gunicorn

gunicorn 'Green Unicorn' is a WSGI HTTP Server for UNIX, fast clients and sleepy applications.
http://www.gunicorn.org
Other
9.86k stars 1.75k forks source link

don't tolerate wrong te headers #3260

Closed benoitc closed 3 months ago

benoitc commented 3 months ago

Just follow the new specification here and accept to introduce a breaking change. Also support multiple encoding on same line.

benoitc commented 3 months ago

cc @pajod @tilgovi

benoitc commented 3 months ago

we will ditch compatibility. It's a major version. Either we follow the spec or not imo. I don't think it will create a major issue especially if people uses a proxy in front of gunicorn , which is encouraged. I prefer the simplicity there/. We need to be clear about the change in the changelog about it.

For "", it's still handled since when you split by "," it can then either be supported (chuunked, identity, deflate, ... or not. Do you mean we need to handle the return diffreently for this case? To which part of the spec does it refers? We can add back this case if needed .