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

bug fix no return Content-Length in the hearders #2291

Merged

Conversation

451973663
Copy link
Contributor

extension-spring6 模块 FastJsonHttpMessageConverter ,修复HTTP返回的请求头中无 Content-Length 字段问题 #2284

@CLAassistant
Copy link

CLAassistant commented Mar 5, 2024

CLA assistant check
All committers have signed the CLA.

@wenshao wenshao requested a review from VictorZeng March 5, 2024 12:02
@wenshao
Copy link
Member

wenshao commented Mar 5, 2024

请签署CLA

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 77.42%. Comparing base (ec1cfb6) to head (5c9ebc2).
Report is 3 commits behind head on main.

Files Patch % Lines
...6/http/converter/FastJsonHttpMessageConverter.java 60.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2291      +/-   ##
============================================
- Coverage     77.42%   77.42%   -0.01%     
+ Complexity    22725    22724       -1     
============================================
  Files           634      634              
  Lines         85224    85225       +1     
  Branches      19322    19322              
============================================
  Hits          65986    65986              
- Misses        11233    11235       +2     
+ Partials       8005     8004       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wenshao wenshao merged commit e5cde6a into alibaba:main Mar 6, 2024
5 checks passed
@451973663 451973663 deleted the fix-bug-extension-spring6-converter branch March 6, 2024 01:30
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