Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

disallow values above 2^60 for Config.MaxIncoming{Uni}Streams #2711

Merged
merged 2 commits into from
Aug 17, 2020

Conversation

marten-seemann
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 11, 2020

Codecov Report

Merging #2711 into master will decrease coverage by 0.02%.
The diff coverage is 66.67%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2711      +/-   ##
==========================================
- Coverage   86.77%   86.74%   -0.02%     
==========================================
  Files         124      124              
  Lines        9893     9905      +12     
==========================================
+ Hits         8584     8592       +8     
- Misses        979      981       +2     
- Partials      330      332       +2     
Impacted Files Coverage Δ
client.go 80.33% <0.00%> (-1.34%) ⬇️
server.go 83.70% <0.00%> (-0.46%) ⬇️
config.go 100.00% <100.00%> (ø)

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 d7c2169...69df425. Read the comment docs.

@marten-seemann marten-seemann merged commit c978b60 into master Aug 17, 2020
@marten-seemann marten-seemann deleted the validate-quic-config branch August 17, 2020 23:59
@aschmahmann aschmahmann mentioned this pull request Sep 22, 2020
72 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants