Catch _all_ multipart NextPart() errors

08c973f0 · Jacob Vosmaer · 0fe5a09e · 08c973f0
Commit 08c973f0 authored Jan 03, 2017 by Jacob Vosmaer
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

internal/upload/rewrite.go internal/upload/rewrite.go +6 -2

No files found.
--- a/internal/upload/rewrite.go
+++ b/internal/upload/rewrite.go
@@ -84,8 +84,11 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te
 	for {
 		p, err := reader.NextPart()
-		if err == io.EOF {
+		if err != nil {
-			break
+			if err == io.EOF {
+				break
+			}
+			return cleanup, err
 		}
 		name := p.FormName()
@@ -105,6 +108,7 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te
 			return cleanup, err
 		}
 	}
 	return cleanup, nil
 }