From 1e7e160d070443147ee38d4de530ce904637a4f3 Mon Sep 17 00:00:00 2001 From: thinkofher Date: Mon, 19 Sep 2022 21:35:04 +0000 Subject: [PATCH] mime/multipart: use %w when wrapping error in NextPart Use "%w" instead of "%v" as format verb for error value in the NextPart method. This way it will be possible to use common go error utilities from std library when parsing from custom io.Readers. This issue was discovered during attempts to use http.Request.ParseMultipartForm together with http.MaxBytesHandler. Change-Id: Idb82510fb536b66b51ed1d943737c4828f07c2f2 GitHub-Last-Rev: 8bc49c945c1b393bcfb4d8df545c93593c9bd7c4 GitHub-Pull-Request: golang/go#55133 Reviewed-on: https://go-review.googlesource.com/c/go/+/431675 Run-TryBot: Ian Lance Taylor Run-TryBot: Damien Neil TryBot-Result: Gopher Robot Auto-Submit: Ian Lance Taylor Reviewed-by: Ian Lance Taylor Reviewed-by: Damien Neil --- src/mime/multipart/multipart.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mime/multipart/multipart.go b/src/mime/multipart/multipart.go index aa05ac8f9c..b3a904f0af 100644 --- a/src/mime/multipart/multipart.go +++ b/src/mime/multipart/multipart.go @@ -360,7 +360,7 @@ func (r *Reader) nextPart(rawPart bool) (*Part, error) { return nil, io.EOF } if err != nil { - return nil, fmt.Errorf("multipart: NextPart: %v", err) + return nil, fmt.Errorf("multipart: NextPart: %w", err) } if r.isBoundaryDelimiterLine(line) {