Skip to content
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

Add support for zstd (Zstandard) encoding #1509

Closed
gaby opened this issue Mar 6, 2023 · 11 comments
Closed

Add support for zstd (Zstandard) encoding #1509

gaby opened this issue Mar 6, 2023 · 11 comments

Comments

@gaby
Copy link
Contributor

gaby commented Mar 6, 2023

Hello, currently fasthttp supports gzip and brotli as encoding when serving compressed content. It also supports them for fasthttp.fs. It would be very beneficial to add support for zstd whicb provides faster decompression rates.

The library used by fashttp already supports zstd. Information here: https://github.com/klauspost/compress/tree/master/zstd#zstd

Tools like curl already support auto-decompression of zstd content since 2020: https://daniel.haxx.se/blog/2020/08/19/curl-7-72-0-more-compression/

@v4rn
Copy link

v4rn commented Mar 19, 2023

@erikdubbelboer new here. can i pick this?

@erikdubbelboer
Copy link
Collaborator

@v4rn yes please!

@gaby
Copy link
Contributor Author

gaby commented Apr 9, 2023

@v4rn You had any luck with this?

@v4rn
Copy link

v4rn commented Apr 11, 2023

@gaby sorry, I had some competing priorities last couple of weeks, just started looking at this. Prioritizing it this week.

@gaby
Copy link
Contributor Author

gaby commented Jun 18, 2023

@v4rn Any luck? thanks!

@gaby
Copy link
Contributor Author

gaby commented Sep 3, 2023

@v4rn Any update on this?

@Max-Cheng
Copy link
Contributor

@gaby Since I'm curious about the internal implementation of fiber, I'm going to start fasthttp from this issue, wish me luck!

@gaby
Copy link
Contributor Author

gaby commented Dec 22, 2023

@Max-Cheng Awesome, yeah main reason i opened this ticket is to eventually add support in Fiber.

@Max-Cheng
Copy link
Contributor

@gaby zstd will be soon. I need more time to test corner cases
#1700

@gaby
Copy link
Contributor Author

gaby commented Jan 20, 2024

@Max-Cheng Awesome! Looking forward

@gaby
Copy link
Contributor Author

gaby commented Feb 21, 2024

Fixed via #1701

@gaby gaby closed this as completed Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants