stream-utils / inflation Goto Github PK

Error: Unsupported Content-Encoding: utf-8
at inflate (/Users/lizuncong/Documents/SheInProjects/mes_wechat/node_modules/inflation/index.js:23:15)
at /Users/lizuncong/Documents/SheInProjects/mes_wechat/node_modules/co-body/lib/json.js:41:18

If the Content-Encoding header is "identity", this module should behave the same way as if the Content-Encoding header was set to an empty string. Currently, an error is thrown.

I can create a PR to fix this if you wish.

Should assert that there is no encoding on the stream handed to the module.

There are no tests, so no way to know if it is really functioning.

The content-encoding header is actually a comma-separated list. We could either support multiple encodings at once, or just assert that there is only one encoding and say we don't (yet) support multiple.

Add the ability to limit the size of the incoming stream. The same limit can apply to the decompressed stream, most likely.

I don't think it should use zlib.Unzip, because I feel like it'll allow people to accidentally send the wrong information to your server without realizing, i.e. they send a gzip stream but say it is deflate and this module will accept it. Later when you put a proxy in front, much breaking will ensue.