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

为cookie添加更多关键字;Response header中Set-Cookie忽略大小写 #1572

Merged
merged 1 commit into from
Jan 15, 2021
Merged

为cookie添加更多关键字;Response header中Set-Cookie忽略大小写 #1572

merged 1 commit into from
Jan 15, 2021

Conversation

happyday517
Copy link
Contributor

header中大小写不匹配还挺常见的,暂时先打个补丁凑合着

@codecov-io
Copy link

Codecov Report

Merging #1572 (33d0eac) into master (4c86bc8) will decrease coverage by 0.00%.
The diff coverage is 58.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1572      +/-   ##
==========================================
- Coverage   48.69%   48.68%   -0.01%     
==========================================
  Files         882      882              
  Lines       37220    37223       +3     
  Branches     8039     8041       +2     
==========================================
- Hits        18124    18123       -1     
- Misses      16683    16684       +1     
- Partials     2413     2416       +3     
Impacted Files Coverage Δ
src/org/nutz/http/Response.java 35.63% <50.00%> (+0.33%) ⬆️
src/org/nutz/http/Cookie.java 37.87% <100.00%> (+4.03%) ⬆️
src/org/nutz/lang/random/ListRandom.java 72.22% <0.00%> (-22.23%) ⬇️
src/org/nutz/mvc/view/UTF8JsonView.java 53.33% <0.00%> (-3.34%) ⬇️
src/org/nutz/lang/socket/Sockets.java 61.90% <0.00%> (-1.91%) ⬇️
src/org/nutz/mvc/view/HttpEnhanceResponse.java 24.13% <0.00%> (-1.73%) ⬇️
src/org/nutz/lang/Lang.java 44.33% <0.00%> (+0.39%) ⬆️

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 4c86bc8...33d0eac. Read the comment docs.

@wendal wendal merged commit b1a1549 into nutzam:master Jan 15, 2021
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.

None yet

3 participants