quic: report error string when peer closes connection

neild · gopherbot · commit 03179ce0330f · 2025-01-07T14:34:40.000-08:00
Change-Id: I2eaf8ef90dc84440e9d38b4d0573c9f6e44f71b0 Reviewed-on: https://go-review.googlesource.com/c/net/+/640798 Reviewed-by: Jonathan Amsterdam <jba@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Damien Neil <dneil@google.com>
diff --git a/quic/errors.go b/quic/errors.go
@@ -121,8 +121,7 @@ type ApplicationError struct {
 }
 
 func (e *ApplicationError) Error() string {
-	// TODO: Include the Reason string here, but sanitize it first.
-	return fmt.Sprintf("AppError %v", e.Code)
+	return fmt.Sprintf("peer closed connection: %v: %q", e.Code, e.Reason)
 }
 
 // Is reports a match if err is an *ApplicationError with a matching Code.

Original file line number	Diff line number	Diff line change
`@@ -121,8 +121,7 @@ type ApplicationError struct {`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`func (e *ApplicationError) Error() string {`
`124`		`- // TODO: Include the Reason string here, but sanitize it first.`
`125`		`- return fmt.Sprintf("AppError %v", e.Code)`
	`124`	`+ return fmt.Sprintf("peer closed connection: %v: %q", e.Code, e.Reason)`
`126`	`125`	`}`
`127`	`126`
`128`	`127`	`// Is reports a match if err is an *ApplicationError with a matching Code.`