From 6ad66d40132737c251622240316fac5c1cfdb141 Mon Sep 17 00:00:00 2001 From: John Rushford Date: Tue, 18 Dec 2018 17:57:40 +0000 Subject: [PATCH] Add additional response code information in responder.go. This would help with issue 3139. --- grove/cache/responder.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/grove/cache/responder.go b/grove/cache/responder.go index 60b9b39a93..929f5190da 100644 --- a/grove/cache/responder.go +++ b/grove/cache/responder.go @@ -16,7 +16,6 @@ package cache import ( "net/http" - "time" "github.com/apache/trafficcontrol/grove/cachedata" "github.com/apache/trafficcontrol/grove/plugin" @@ -96,7 +95,7 @@ func (r *Responder) Do() { // TODO move plugins.BeforeRespond here? How do we distinguish between success, and know to set headers? r.OriginReqSuccess? bytesSent, err := r.F() if err != nil { - log.Errorln(time.Now().Format(time.RFC3339Nano) + " " + r.Req.RemoteAddr + " " + r.Req.Method + " " + r.Req.RequestURI + ": responding: " + err.Error()) + log.Errorf("%s %s %s %v : responding: %v", r.Req.RemoteAddr, r.Req.Method, r.Req.RequestURI, r.ResponseCode, err.Error()) } web.TryFlush(r.W) // TODO remove? Let plugins do it, if they need to?