alexvanin/certificates
1
0
Fork 0
forked from TrueCloudLab/certificates
Code Pull requests Activity

Remove unnecessary log entries

Browse source 
These log entries add CodeQL warnings and are not necessary because
our default http.ResponseWriter allows adding log entries.
This commit is contained in:
Mariano Cano 2022-08-11 18:14:36 -07:00
parent 759aa26a57
commit 2db15e4eb5
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
api/log/log.go
View file

@ -3,7 +3,6 @@ package log
import ( import (
"fmt" "fmt"
"log"
"net/http" "net/http"
"os" "os"
@ -28,8 +27,6 @@ type StackTracedError interface {
func Error(rw http.ResponseWriter, err error) { func Error(rw http.ResponseWriter, err error) {
rl, ok := rw.(logging.ResponseLogger) rl, ok := rw.(logging.ResponseLogger)
if !ok { if !ok {
log.Println(err)
return return
} }
@ -72,8 +69,6 @@ func EnabledResponse(rw http.ResponseWriter, v interface{}) {
rl.WithFields(map[string]interface{}{ rl.WithFields(map[string]interface{}{
"response": out, "response": out,
}) })
} else {
log.Println(out)
} }
} }
} }