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

improve string representation of timeout errors

Browse Source
This commit is contained in:
Marten Seemann
2021-03-23 14:37:49 +08:00
parent 81d16a9903
commit afbb2b7b6a
2 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
internal/qerr/quic_error.go
View File

@@ -65,9 +65,14 @@ func (e *QuicError) Error() string {
} }
return fmt.Sprintf("Application error %#x: %s", uint64(e.ErrorCode), e.ErrorMessage) return fmt.Sprintf("Application error %#x: %s", uint64(e.ErrorCode), e.ErrorMessage)
} }
str := e.ErrorCode.String() var str string
if e.FrameType != 0 { if e.isTimeout {
str += fmt.Sprintf(" (frame type: %#x)", e.FrameType) str = "Timeout"
} else {
str = e.ErrorCode.String()
if e.FrameType != 0 {
str += fmt.Sprintf(" (frame type: %#x)", e.FrameType)
}
} }
msg := e.ErrorMessage msg := e.ErrorMessage
if len(msg) == 0 { if len(msg) == 0 {

2
internal/qerr/quic_error_test.go
View File

@@ -33,7 +33,7 @@ var _ = Describe("QUIC Transport Errors", func() {
It("has a string representation for timeout errors", func() { It("has a string representation for timeout errors", func() {
err := NewTimeoutError("foobar") err := NewTimeoutError("foobar")
Expect(err.Timeout()).To(BeTrue()) Expect(err.Timeout()).To(BeTrue())
Expect(err.Error()).To(Equal("NO_ERROR: foobar")) Expect(err.Error()).To(Equal("Timeout: foobar"))
}) })
Context("crypto errors", func() { Context("crypto errors", func() {