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

resolve response is gzip in internalMiddlewareServerTracing #3055

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

ltp217
Copy link
Contributor

@ltp217 ltp217 commented Oct 17, 2023

resolve response is gzip in internalMiddlewareServerTracing
the issue is: #2792

@fainc
Copy link
Contributor

fainc commented Oct 17, 2023

755 756 行重复设置 header,刚好在做自定义reponse中间件,copy一下 😍

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Lines 755 and 756 repeatedly set the header. I am just doing custom response middleware, copy it 😍

@gqcn gqcn merged commit 34e5223 into gogf:master Oct 17, 2023
12 checks passed
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.

4 participants