Skip to content

[type: refactor] remove gzip when the request headers exist, to fix modifyResponse Plugin and other exchange mutate plugin.#2301

Merged
yu199195 merged 2 commits intoapache:masterfrom
openingo:fix-gzip-encoding
Nov 4, 2021
Merged

[type: refactor] remove gzip when the request headers exist, to fix modifyResponse Plugin and other exchange mutate plugin.#2301
yu199195 merged 2 commits intoapache:masterfrom
openingo:fix-gzip-encoding

Conversation

@qicz
Copy link
Copy Markdown
Member

@qicz qicz commented Nov 4, 2021

No description provided.

qicz added 2 commits November 4, 2021 16:40
…odifyResponse Plugin and other exchange mutate plugin.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 4, 2021

Codecov Report

Merging #2301 (7dcc05f) into master (29a4ffa) will decrease coverage by 0.00%.
The diff coverage is 25.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2301      +/-   ##
============================================
- Coverage     58.46%   58.45%   -0.01%     
+ Complexity     3968     3966       -2     
============================================
  Files           626      626              
  Lines         18053    18056       +3     
  Branches       1711     1712       +1     
============================================
  Hits          10555    10555              
- Misses         6638     6641       +3     
  Partials        860      860              
Impacted Files Coverage Δ
...henyu/plugin/httpclient/NettyHttpClientPlugin.java 56.36% <16.66%> (-3.64%) ⬇️
...ache/shenyu/plugin/httpclient/WebClientPlugin.java 84.44% <20.00%> (-8.06%) ⬇️
.../apache/shenyu/plugin/response/ResponsePlugin.java 77.77% <100.00%> (+15.27%) ⬆️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 64.21% <0.00%> (+1.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29a4ffa...7dcc05f. Read the comment docs.

@yu199195 yu199195 added this to the 2.4.2 milestone Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants