4194: use same log statement for successes as well as errors.

[arvados.git] / services / keepstore / logging_router.go

diff --git a/services/keepstore/logging_router.go b/services/keepstore/logging_router.go
index 6927019fd917ad69b90f8367ce81a2dce96e8251..fd4e23467d63d8e91cece66d8e5cd46be7c35cfb 100644 (file)
--- a/services/keepstore/logging_router.go
+++ b/services/keepstore/logging_router.go
@@ -21,7 +21,7 @@ func (loggingWriter *LoggingResponseWriter) WriteHeader(code int) {
 }
 
 func (loggingWriter *LoggingResponseWriter) Write(data []byte) (int, error) {
-       loggingWriter.Length = len(data)
+       loggingWriter.Length += len(data)
        return loggingWriter.ResponseWriter.Write(data)
 }
 
@@ -37,9 +37,5 @@ func MakeLoggingRESTRouter() *LoggingRESTRouter {
 func (loggingRouter *LoggingRESTRouter) ServeHTTP(resp http.ResponseWriter, req *http.Request) {
        loggingWriter := LoggingResponseWriter{200, 0, resp}
        loggingRouter.router.ServeHTTP(&loggingWriter, req)
-       if loggingWriter.Status == 200 {
-               log.Printf("[%s] %s %s %d %d", req.RemoteAddr, req.Method, req.URL.Path[1:], loggingWriter.Status, loggingWriter.Length)
-       } else {
-               log.Printf("[%s] %s %s %d", req.RemoteAddr, req.Method, req.URL.Path[1:], loggingWriter.Status)
-       }
+       log.Printf("[%s] %s %s %d %d", req.RemoteAddr, req.Method, req.URL.Path[1:], loggingWriter.Status, loggingWriter.Length)
 }