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

use the correct error code for crypto stream errors

Browse Source
This commit is contained in:
Marten Seemann
2020-05-14 17:12:07 +07:00
parent 1cd1c3f54d
commit df06ee5e37
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
crypto_stream.go
View File

@@ -1,7 +1,6 @@
package quic package quic
import ( import (
"errors"
"fmt" "fmt"
"io" "io"
@@ -81,7 +80,7 @@ func (s *cryptoStreamImpl) GetCryptoData() []byte {
func (s *cryptoStreamImpl) Finish() error { func (s *cryptoStreamImpl) Finish() error {
if s.queue.HasMoreData() { if s.queue.HasMoreData() {
return errors.New("encryption level changed, but crypto stream has more data to read") return qerr.NewError(qerr.ProtocolViolation, "encryption level changed, but crypto stream has more data to read")
} }
s.finished = true s.finished = true
return nil return nil

2
crypto_stream_test.go
View File

@@ -97,7 +97,7 @@ var _ = Describe("Crypto Stream", func() {
Offset: 10, Offset: 10,
})).To(Succeed()) })).To(Succeed())
err := str.Finish() err := str.Finish()
Expect(err).To(MatchError("encryption level changed, but crypto stream has more data to read")) Expect(err).To(MatchError("PROTOCOL_VIOLATION: encryption level changed, but crypto stream has more data to read"))
}) })
It("works with reordered data", func() { It("works with reordered data", func() {