Skip to content

Add support for zstd#250

Open
polys wants to merge 1 commit intoexpressjs:masterfrom
polys:zstd-support
Open

Add support for zstd#250
polys wants to merge 1 commit intoexpressjs:masterfrom
polys:zstd-support

Conversation

@polys
Copy link

@polys polys commented Aug 12, 2025

Adds support for zstd, but keeps brotli as preferred.

Resolves #217

@Phillip9587
Copy link
Member

I think we decided to wait for zstd to become stable in Node.js before using it:

@polys
Copy link
Author

polys commented Aug 12, 2025

I think we decided to wait for zstd to become stable in Node.js before using it:

Thank you, that's perfectly fine :)
I did the work for a little experiment of mine and thought I'd open the PR in case it's useful when the time is right.

@bjohansebas bjohansebas mentioned this pull request Aug 28, 2025
kirillgroshkov added a commit to NaturalCycles/js-libs that referenced this pull request Dec 21, 2025
@DienoX
Copy link

DienoX commented Feb 11, 2026

Isn't it that time already?

@Phillip9587
Copy link
Member

Zsdt is still experimental as of Node.js 25.6.1: https://nodejs.org/api/zlib.html#class-zlibzstdcompress

@bjohansebas
Copy link
Member

While it’s still experimental, it’s already being adopted by several important packages like fastify/compress, webpack-bundle-analyzer, and likely many others. I think it would be a good idea to include it. There haven’t been many issues reported about this, and the existing ones seem more like Node.js-related problems rather than ours.

That said, I do wonder whether this should probably be a major version or not. I’m asking because when I implemented Brotli in compression as a minor release, issues were reported by people who couldn’t control which encodings were accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support zstd

4 participants

Comments