compression

compression copied to clipboard

fix: call underlying implementation properly

33

related issue [#5431](https://github.com/sveltejs/kit/issues/5431)

fernandolguevara

Use headersSent instead of _header

4

As discussed in #127 this PR now includes the changes that **do** break node v0.8 and has tests. I did not remove the travis config for v0.8 since I'm not...

maritz

Bugfix/use write head instead of implicit header

12

updated PR with rebase from express/compress:master to handle #128

Icehunter

Fix usage of undocumented _implicitHeader

38

As discussed in #127 this PR now only includes the changes that should not break node v0.8 and has tests. Sadly supertest does not support http2 yet, so I had...

maritz

Crash when compressing characters like ū

1

I tried to compress `"Chūnin"` and this compression function crashed. Tried to use `btoa("Chūnin")` but `btoa` doesn't support non-ASCII either.

tsukinoko-kun

Added support for brotli ('br') content-encoding

58

https://github.com/expressjs/body-parser/pull/403 https://medium.com/oyotech/how-brotli-compression-gave-us-37-latency-improvement-14d41e50fee4 https://caniuse.com/#feat=brotli

danielgindi

feat: Adding support for brolti

9

Fixes #71 ## TL;DR There has been several attempts (#172, #158) to add [brotli compression](https://en.wikipedia.org/wiki/Brotli) support to this library and so far none have been successfully merged (for various reasons)....

nicksrandall

"drain" event listener leak when unpiping to response

2

This is sort of a weird scenario, but this test case fails: ```js it('should clean up event listeners when res is unpiped to', function (done) { var listenerCount var server...

zbjornson

Other Express-related packages like `body-parser` and `raw-body` require `[email protected]` so upgrading this dependency in `compression` saves a copy of `bytes` on all installations that combine `express` and `compression`.

jorrit

feat: support brotli

7

clarkdo