Commit aa4addd7 authored by Nick Thomas's avatar Nick Thomas

Merge branch 'multipart-nextpart-error' into 'master'

Catch _all_ multipart NextPart() errors

Closes #89

See merge request !108
parents f151982c 08c973f0
...@@ -87,9 +87,12 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te ...@@ -87,9 +87,12 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te
for { for {
p, err := reader.NextPart() p, err := reader.NextPart()
if err != nil {
if err == io.EOF { if err == io.EOF {
break break
} }
return cleanup, err
}
name := p.FormName() name := p.FormName()
if name == "" { if name == "" {
...@@ -108,6 +111,7 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te ...@@ -108,6 +111,7 @@ func rewriteFormFilesFromMultipart(r *http.Request, writer *multipart.Writer, te
return cleanup, err return cleanup, err
} }
} }
return cleanup, nil return cleanup, nil
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment