mirrors/uquic
1
0
Fork 0
mirror of https://github.com/refraction-networking/uquic.git synced 2025-04-05 13:17:36 +03:00
Code Issues Projects Releases Packages Wiki Activity

don't accept transport parameters in the NewSessionTicket message

Browse source 
This was recently removed from the WG draft.
This commit is contained in:
Marten Seemann 2018-02-08 11:09:42 +08:00
parent d71850eb2f
commit f0eb5a60f1
2 changed files with 1 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

7
internal/handshake/tls_extension_handler_client.go
View file

@ -63,17 +63,12 @@ func (h *extensionHandlerClient) Receive(hType mint.HandshakeType, el *mint.Exte
return err return err
} }
if hType != mint.HandshakeTypeEncryptedExtensions && hType != mint.HandshakeTypeNewSessionTicket { if hType != mint.HandshakeTypeEncryptedExtensions {
if found { if found {
return fmt.Errorf("Unexpected QUIC extension in handshake message %d", hType) return fmt.Errorf("Unexpected QUIC extension in handshake message %d", hType)
} }
return nil return nil
} }
if hType == mint.HandshakeTypeNewSessionTicket {
// the extension it's optional in the NewSessionTicket message
// TODO: handle this
return nil
}
// hType == mint.HandshakeTypeEncryptedExtensions // hType == mint.HandshakeTypeEncryptedExtensions
if !found { if !found {

8
internal/handshake/tls_extension_handler_client_test.go
View file

@ -100,14 +100,6 @@ var _ = Describe("TLS Extension Handler, for the client", func() {
Expect(err).ToNot(HaveOccurred()) Expect(err).ToNot(HaveOccurred())
}) })
// TODO: fix this when implementing the NewSessionTicket
It("ignors the TransportParameters in the NewSessionTicket message", func() {
err := el.Add(fakeBody)
Expect(err).ToNot(HaveOccurred())
err = handler.Receive(mint.HandshakeTypeNewSessionTicket, &el)
Expect(err).ToNot(HaveOccurred())
})
It("errors when it can't parse the TransportParameters", func() { It("errors when it can't parse the TransportParameters", func() {
err := el.Add(fakeBody) err := el.Add(fakeBody)
Expect(err).ToNot(HaveOccurred()) Expect(err).ToNot(HaveOccurred())