vbatts/registry
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Ensure we log io.Copy errors and bytes copied/total in uploads

Browse source 
Signed-off-by: Tony Holdstock-Brown <tony@docker.com>
This commit is contained in:
Tony Holdstock-Brown 2016-04-04 17:18:09 -07:00
parent 20fa47886d
commit 25c5efdef9
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
registry/handlers/helpers.go
View file

@ -46,7 +46,11 @@ func copyFullPayload(responseWriter http.ResponseWriter, r *http.Request, destWr
// instead of showing 0 for the HTTP status. // instead of showing 0 for the HTTP status.
responseWriter.WriteHeader(499) responseWriter.WriteHeader(499)
ctxu.GetLogger(context).Error("client disconnected during " + action) ctxu.GetLoggerWithFields(context, map[interface{}]interface{}{
"error": err,
"copied": copied,
"contentLength": r.ContentLength,
}, "error", "copied", "contentLength").Error("client disconnected during " + action)
return errors.New("client disconnected") return errors.New("client disconnected")
default: default:
} }