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.

Sign up for GitHub

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

Jump to bottom

Made chunked encoding HTTP header check stricter #6305

Merged
merged 2 commits into from Nov 13, 2021
Merged

Made chunked encoding HTTP header check stricter #6305

merged 2 commits into from Nov 13, 2021

Conversation

asvetlov
Copy link
Member

No description provided.

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Nov 13, 2021
@asvetlov asvetlov added backport-3.8 backport-3.9 Trigger automatic backporting to the 3.9 release branch by Patchback robot and removed bot:chronographer:provided There is a change note present in this PR labels Nov 13, 2021
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Nov 13, 2021
@codecov
Copy link

codecov bot commented Nov 13, 2021 •
edited

Codecov Report

Merging #6305 (8ae8932) into master (33b630d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6305   +/-   ##
=======================================
  Coverage   93.30%   93.30%           
=======================================
  Files         103      103           
  Lines       30362    30366    +4     
  Branches     2729     2730    +1     
=======================================
+ Hits        28328    28333    +5     
  Misses       1857     1857           
+ Partials      177      176    -1
Flag Coverage Δ
unit 93.22% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
aiohttp/http_parser.py 92.66% <100.00%> (+0.20%) ⬆️
tests/test_http_parser.py 99.06% <100.00%> (+<0.01%) ⬆️

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 33b630d...8ae8932. Read the comment docs.

@asvetlov asvetlov merged commit 7f32513 into master Nov 13, 2021
@asvetlov asvetlov deleted the strict-chunked-test branch November 13, 2021 11:54
@patchback
Copy link
Contributor

patchback bot commented Nov 13, 2021 •
edited

Backport to 3.8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.8/7f3251359b2ece0641acd0ccc5515dafaab5d8ed/pr-6305

Backported as #6306

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 13, 2021
@asvetlov
Made chunked encoding HTTP header check stricter (#6305)
6bb82b6 
(cherry picked from commit 7f32513)
@patchback patchback bot mentioned this pull request Nov 13, 2021
Merged
@patchback
Copy link
Contributor

patchback bot commented Nov 13, 2021 •
edited

Backport to 3.9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.9/7f3251359b2ece0641acd0ccc5515dafaab5d8ed/pr-6305

Backported as #6307

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 13, 2021
@asvetlov
Made chunked encoding HTTP header check stricter (#6305)
3e599a8 
(cherry picked from commit 7f32513)
@patchback patchback bot mentioned this pull request Nov 13, 2021
Merged
asvetlov added a commit that referenced this pull request Nov 13, 2021
@patchback @asvetlov
Made chunked encoding HTTP header check stricter (#6305) (#6306)
8457ffd 
(cherry picked from commit 7f32513)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
asvetlov added a commit that referenced this pull request Nov 13, 2021
@patchback @asvetlov
Made chunked encoding HTTP header check stricter (#6305) (#6307)
4443094 
(cherry picked from commit 7f32513)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport-3.9 Trigger automatic backporting to the 3.9 release branch by Patchback robot bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@asvetlov