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 ResponseHeader.contentEncoding #456
feat: add ResponseHeader.contentEncoding #456
Conversation
@li-jin-gou plz approve running workflows & Review |
Codecov ReportBase: 69.53% // Head: 69.61% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #456 +/- ##
===========================================
+ Coverage 69.53% 69.61% +0.08%
===========================================
Files 93 93
Lines 8810 8834 +24
===========================================
+ Hits 6126 6150 +24
Misses 2320 2320
Partials 364 364
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@li-jin-gou I'd add some ut plz approve running workflows again |
@li-jin-gou plz review it ,thanks |
Please change the title to the standard format |
|
PTAL |
最近新冠中,康复了之后我做一下修改发自我的 iPhone在 2022年12月15日,18:53,skyenought ***@***.***> 写道:
@Skyenought commented on this pull request.
In pkg/protocol/header.go:
@@ -224,6 +225,7 @@ func (h *ResponseHeader) CopyTo(dst *ResponseHeader) {
dst.statusCode = h.statusCode
dst.contentLength = h.contentLength
dst.contentLengthBytes = append(dst.contentLengthBytes[:0], h.contentLengthBytes...)
+ dst.contentEncoding = append(dst.contentEncoding, h.contentEncoding...)
dst.Reset() 中应该有这样的逻辑吧
h.contentEncoding = h.contentEncoding[:0]
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
This reverts commit 46975c4.
…nto feat_AddHeaderContentEncoding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx~
What type of PR is this?
feat
Check the PR title.
(Optional) Translate the PR title into Chinese.
为ResponseHeader 添加 contentEncoding 及相关方法
(Optional) More detail description for this PR(en: English/zh: Chinese).
en: Adding a new field contentEncoding into ResponseHeader
Also fix Referer and Tests
zh(optional): 为ResponseHeader 添加一个字段 contentEncoding 同时修改测试以及其他方法中的引用
Which issue(s) this PR fixes:
Fixes #449
refer to valyala/fasthttp#1311