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

feat: add gzip for client #4

Merged
merged 24 commits into from
Mar 8, 2023

Conversation

zstone12
Copy link
Collaborator

@zstone12 zstone12 commented Feb 15, 2023

What type of PR is this?

Add gzip for client
(Now in demo stage...

  • Optimised code structure
  • Add unit tests
  • Readme

What this PR does / why we need it (English/Chinese):

Which issue(s) this PR fixes:

@zstone12 zstone12 changed the title [WIP]add gzip for client feat: add gzip for client Feb 20, 2023
@li-jin-gou
Copy link
Collaborator

ci need to solve

@zstone12 zstone12 force-pushed the feature/gzip_for_client branch from 49f2962 to c1f5fb7 Compare February 24, 2023 14:51
@zstone12
Copy link
Collaborator Author

ci need to solve

done

@zstone12 zstone12 force-pushed the feature/gzip_for_client branch from ba8bcdc to da49ef0 Compare February 25, 2023 01:24
middleware.go Outdated Show resolved Hide resolved
middleware.go Outdated Show resolved Hide resolved
middleware.go Outdated Show resolved Hide resolved
middleware_test.go Outdated Show resolved Hide resolved
@zstone12 zstone12 force-pushed the feature/gzip_for_client branch from 77754bd to 947dfe7 Compare March 1, 2023 14:21
@li-jin-gou
Copy link
Collaborator

ci need to solve

@zstone12 zstone12 force-pushed the feature/gzip_for_client branch from 4950b1f to c6217d7 Compare March 7, 2023 03:32
@zstone12 zstone12 closed this Mar 7, 2023
@zstone12 zstone12 reopened this Mar 7, 2023
@zstone12
Copy link
Collaborator Author

zstone12 commented Mar 7, 2023

ci need to solve

done.tks

@li-jin-gou li-jin-gou merged commit faffd46 into hertz-contrib:main Mar 8, 2023
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.

5 participants