-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Bun 1.3.3 added CompressionStream/DecompressionStream
#28508
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
CompressionStream & DecompressionStreamCompressionStream/DecompressionStream
caugner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments above.
(Your branch cannot be modified by maintainers so I'm unable to remove the tags my self. Note: It's recommended to allow maintainers to edit.)
Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>
Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>
Oh yeah I forget github permissions are weird - I generally like using an org for prs because it separates them from me personally (ie simplifies my repos shown)
Yeah fair, I'll remove it. I more or less copied the brotli implementation which also I believe is non-standard right now (but does have slightly better adoption) browser-compat-data/api/CompressionStream.json Lines 92 to 97 in 1a1e05f
|
Summary
Bun 1.3.3 added
CompressionStream&DecompressionStreamfor the standard "gzip", "deflate" and "deflate-raw" formats, and also supports "brotli" and "zstd".As no other platforms have zstd yet I added it, but I can also remove if its meant to have more usage first.
Test results and supporting details
Related issues