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.

Sign up for GitHub

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

Jump to bottom

fix: use gzip compression only for /write endpoint #851

Merged
merged 3 commits into from Jul 5, 2022
Merged

fix: use gzip compression only for /write endpoint #851

merged 3 commits into from Jul 5, 2022

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Jul 4, 2022 •
edited

Closes #848

Proposed Changes

Only the request to /write endpoint should be compressed by GZIP.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • mvn test completes successfully
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@bednar bednar requested a review from majst01 July 4, 2022 14:44
@bednar bednar marked this pull request as ready for review July 4, 2022 14:44
@majst01 majst01 merged commit 99dbf49 into master Jul 5, 2022
@majst01 majst01 deleted the query-gzip branch July 5, 2022 05:00
@majst01
Copy link
Collaborator

majst01 commented Jul 5, 2022

@bednar thanks for taking care, can you create a release out of current master ?

@bednar
Copy link
Contributor Author

bednar commented Jul 5, 2022

@majst01, I am out of office tomorrow. I will release the client at Thursday.

@majst01
Copy link
Collaborator

majst01 commented Jul 6, 2022

@majst01, I am out of office tomorrow. I will release the client at Thursday.

OK

@bednar bednar added this to the 2.23 milestone Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@majst01 majst01 majst01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.23
2 participants
@bednar @majst01