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

CompressionLayer add zstd support #319

Closed
beckend opened this issue Jan 8, 2023 · 1 comment · Fixed by #322
Closed

CompressionLayer add zstd support #319

beckend opened this issue Jan 8, 2023 · 1 comment · Fixed by #322
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. E-help-wanted Call for participation: Help is requested to fix this issue. E-medium Call for participation: Experience needed to fix: Medium / intermediate

Comments

@beckend
Copy link

beckend commented Jan 8, 2023

Feature Request

Motivation

It's new, but it's starting to pick up.
Here is a curl request:

curl --compressed http://whatever.com/my/endpoint
> GET /my/endpoint HTTP/1.1
> Host: whatever.com
> User-Agent: curl/7.87.0
> Accept: */*
> Accept-Encoding: deflate, gzip, br, zstd

It's also available in envoy proxy serving such requests.
https://github.com/envoyproxy/envoy/tree/main/examples/zstd

@davidpdrsn
Copy link
Member

That should be quite straightforward seeing as async-compression already supports it.

@davidpdrsn davidpdrsn added C-feature-request Category: A feature request, i.e: not implemented / a PR. E-help-wanted Call for participation: Help is requested to fix this issue. E-medium Call for participation: Experience needed to fix: Medium / intermediate labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature-request Category: A feature request, i.e: not implemented / a PR. E-help-wanted Call for participation: Help is requested to fix this issue. E-medium Call for participation: Experience needed to fix: Medium / intermediate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants