gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

don't call ClosedConnection when closing for version negotiation

Browse Source
This commit is contained in:
Marten Seemann
2021-03-22 09:58:26 +08:00
parent c30a45ef6f
commit 22daaf4536
2 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
session.go
View File

@@ -1436,7 +1436,7 @@ func (s *session) handleCloseError(closeErr closeError) {
s.datagramQueue.CloseWithError(quicErr) s.datagramQueue.CloseWithError(quicErr)
} }
if s.tracer != nil { if s.tracer != nil && !errors.Is(closeErr.err, errCloseForRecreating{}) {
var resetErr statelessResetErr var resetErr statelessResetErr
var vnErr errVersionNegotiation var vnErr errVersionNegotiation
switch { switch {

1
session_test.go
View File

@@ -2608,7 +2608,6 @@ var _ = Describe("Client Session", func() {
ContainElement(protocol.VersionNumber(1337)), ContainElement(protocol.VersionNumber(1337)),
)) ))
}) })
tracer.EXPECT().ClosedConnection(gomock.Any())
cryptoSetup.EXPECT().Close() cryptoSetup.EXPECT().Close()
Expect(sess.handlePacketImpl(getVNP(4321, 1337))).To(BeFalse()) Expect(sess.handlePacketImpl(getVNP(4321, 1337))).To(BeFalse())
var err error var err error