From 2db15e4eb5f0d945c1cf8bea3618243bc100614f Mon Sep 17 00:00:00 2001 From: Mariano Cano Date: Thu, 11 Aug 2022 18:14:36 -0700 Subject: [PATCH] Remove unnecessary log entries These log entries add CodeQL warnings and are not necessary because our default http.ResponseWriter allows adding log entries. --- api/log/log.go | 5 ----- 1 file changed, 5 deletions(-) diff --git a/api/log/log.go b/api/log/log.go index cb31410b..e5c8c45a 100644 --- a/api/log/log.go +++ b/api/log/log.go @@ -3,7 +3,6 @@ package log import ( "fmt" - "log" "net/http" "os" @@ -28,8 +27,6 @@ type StackTracedError interface { func Error(rw http.ResponseWriter, err error) { rl, ok := rw.(logging.ResponseLogger) if !ok { - log.Println(err) - return } @@ -72,8 +69,6 @@ func EnabledResponse(rw http.ResponseWriter, v interface{}) { rl.WithFields(map[string]interface{}{ "response": out, }) - } else { - log.Println(out) } } }