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

feature: support HTTP compression #1684

Merged
merged 1 commit into from
Nov 3, 2023
Merged

feature: support HTTP compression #1684

merged 1 commit into from
Nov 3, 2023

Conversation

Kree0
Copy link
Contributor

@Kree0 Kree0 commented Oct 31, 2023

Required checklist

  • Sample config files updated (config,meta/config and default config)
  • If there are user-facing changes, the documentation needs to be updated prior to approving the PR( Link )
  • If there are any breaking changes to public APIs, please add the api change label.
  • Signed CLA (if not already signed)

Which issue does this PR close?

Related #.

Rationale for this change

Are there any user-facing changes?

Support compression for HTTP request and response.

The request header can specify: accept-encoding, content-encoding.

Accept-encoding: specify the desired response's encoding.

Content-encoding: specify the request's encoding.

Supported encoding: gzip, deflate, br, zstd.

client/src/ctx.rs Outdated Show resolved Hide resolved
@Kree0 Kree0 force-pushed the http branch 4 times, most recently from 42d6060 to b5ee8a2 Compare November 2, 2023 10:51
Copy link
Contributor

@roseboy-liu roseboy-liu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add test case for feature

roseboy-liu
roseboy-liu previously approved these changes Nov 3, 2023
@Kree0
Copy link
Contributor Author

Kree0 commented Nov 3, 2023

add test case for feature

Added.

@zipper-meng
Copy link
Member

add test case for feature

Added.

At e2e_test/src/reliant/http_api_tests.rs

@zipper-meng zipper-meng merged commit 99d25c3 into cnosdb:main Nov 3, 2023
7 checks passed
@Kree0 Kree0 deleted the http branch November 9, 2023 01:45
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.

None yet

3 participants