go/echo
go/echo
1
0
Fork 0
mirror of https://github.com/labstack/echo.git synced 2025-05-31 23:19:42 +02:00
Code Issues Releases Activity

Improved error handling of multipart requests

Browse Source 
standard/request: return the error of the http request multipart parsing
call
test/request: similar functionality as the standard/request
This commit is contained in:
rlcomte 2016-03-18 09:42:10 +01:00
parent 3946a1839c
commit cc86fa2232
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
engine/standard/request.go
View File

@ -100,8 +100,8 @@ func (r *Request) FormFile(name string) (*multipart.FileHeader, error) {
// MultipartForm implements `engine.Request#MultipartForm` method. // MultipartForm implements `engine.Request#MultipartForm` method.
func (r *Request) MultipartForm() (*multipart.Form, error) { func (r *Request) MultipartForm() (*multipart.Form, error) {
r.Request.ParseMultipartForm(32 << 20) // 32 MB err := r.Request.ParseMultipartForm(32 << 20) // 32 MB
return r.Request.MultipartForm, nil return r.Request.MultipartForm, err
} }
func (r *Request) reset(req *http.Request, h engine.Header, u engine.URL) { func (r *Request) reset(req *http.Request, h engine.Header, u engine.URL) {

4
test/request.go
View File

@ -94,8 +94,8 @@ func (r *Request) FormFile(name string) (*multipart.FileHeader, error) {
} }
func (r *Request) MultipartForm() (*multipart.Form, error) { func (r *Request) MultipartForm() (*multipart.Form, error) {
m := r.request.MultipartForm err := r.request.ParseMultipartForm(32 << 20) // 32 MB
return m, nil return r.request.MultipartForm, err
} }
func (r *Request) reset(req *http.Request, h engine.Header, u engine.URL) { func (r *Request) reset(req *http.Request, h engine.Header, u engine.URL) {