inflation

inflation copied to clipboard

Support Brotli decompression (content-encoding: br)

1

Adds backwards-compatible support for Brotli decompression for versions of Node that natively support Brotli (10.16.0+). For reference, Node LTS is soon to be 14.15.0+ only. If the version of Node...

ide

feat: now supports 'content-encoding:utf-8' (closes #16)

1

fix #16 Added a new test for it, and all tests pass.

jingxinxin

if I set the ctx.request.headers['content-encoding'] = 'utf-8'，there will be an error。

1

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

lizuncong

Docs were taken literally from the readme.

remcohaszing

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

dougwilson

Don't use zlib.Unzip

1

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...

dougwilson

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

dougwilson