Feature\#89 - Per route options for compression

[Svjard]

Checklist

• run npm run test and npm run benchmark
• tests and/or benchmarks are included
• documentation is changed or added
• commit message and code follows Code of conduct

[Svjard]

Feature for #89.

Took a first pass at this welcome feedback on it. In particular:

1. Removed the decorate call to instead have compress() flow from the start of the request on the reply. Because each route can be configured differently was hard to determine the best approach from the previous approach.
2. Better tests for per route, have a minimal set but open to writing more if you have some good ones I can add.
3. The package.json is pointing to master, I know this needs to change but waiting for the next Fastify publish.

[Eomm]

Great!

[Eomm]

It should be released in 2.11 👍

[marc-fisher-q2e]

Done, thanks

[mcollina]

https://github.com/fastify/fastify-compress/pull/92/files#diff-168726dbe96b3ce427e7fedce31bb0bcR314

Can you bump this to fastify v2.11? That's the one you'll need for this fix.

[mcollina]

(this will be a semver-major change).

[mcollina]

Good work! Don't forget to change

fastify-compress/index.js

Line 250 in b2a1995

fastify: '>=1.3.0',

.

[mcollina]

Given that params will not change, can you cache the output of compress(params) outside of the onRequest hook?

[Svjard]

👍

[mcollina]

LGTM