-
Notifications
You must be signed in to change notification settings - Fork 525
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
Support incorrect deflate content encoding that browser is supporting #1019
Comments
PR welcome |
The |
chrome code that inflecting the data: https://source.chromium.org/chromium/chromium/src/+/main:net/filter/gzip_source_stream.cc;l=134 |
I'm -1 on this one. https://datatracker.ietf.org/doc/html/rfc7230#section-4.2.2
|
But can we just add a flag, just like the |
@mcollina wdyt? node-fetch has support for this https://github.com/node-fetch/node-fetch/blob/8b54ab4509b541dadcd2d66aa5cf0995ad54ac15/src/index.js#L258-L270 |
I think those http servers are still around, so we should unfortunately support it. I would mark this as "good first issue", so if somebody needs this they know they can implement it. |
This issue is the same as the one linked below. |
This can be closed. I had a wrong issue linked with my PR #2650 |
Look at the code.
deflateRawSync
is not conform to the http standard. But most browser can correctly open it.AboutError
is not clearly.undici.request
not supportdeflate
encoding?The text was updated successfully, but these errors were encountered: