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
fix handling of ACK frames serialized after CRYPTO frames #4018
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't feel like I understand the protocol thoroughly enough to actually review this, but the basic logic LGTM. Nice and simple fix.
71ce36b
to
b47ab96
Compare
Thanks for the review @mholt! I really hope this is the last critical bug in the v0.37 release line. Turns out that switching out the underlying TLS stack (that's in fact what we did, given that the crypto/tls API is completely different (better!) from what we had in qtls) is a minefield. |
Codecov Report
@@ Coverage Diff @@
## master #4018 +/- ##
==========================================
+ Coverage 82.76% 83.10% +0.34%
==========================================
Files 147 147
Lines 14764 14872 +108
==========================================
+ Hits 12219 12359 +140
+ Misses 2042 2012 -30
+ Partials 503 501 -2
|
b47ab96
to
c239066
Compare
@@ -1724,6 +1743,7 @@ func (s *connection) checkTransportParameters(params *wire.TransportParameters) | |||
|
|||
func (s *connection) applyTransportParameters() { | |||
params := s.peerParams | |||
fmt.Println("apply transport parameters", s.config.MaxIdleTimeout, params.MaxIdleTimeout) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leftover debug statement. Merged via this PR, then fix using force-push on master. Sorry for that.
Thank you @marten-seemann -- absolutely understandable and no worries :) We have released Caddy 2.7.3 with the patches. |
No description provided.