Skip to content

Commit d826f7f

Browse files
committed
Fix gosec findings in main.go and json.go
1 parent a580f80 commit d826f7f

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

json.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,5 +30,8 @@ func respondWithJSON(w http.ResponseWriter, code int, payload interface{}) {
3030
return
3131
}
3232
w.WriteHeader(code)
33-
w.Write(dat)
33+
_, err = w.Write(dat)
34+
if err != nil {
35+
log.Printf("Error writing data to response writer: %s", err)
36+
}
3437
}

main.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import (
77
"log"
88
"net/http"
99
"os"
10+
"time"
1011

1112
"github.com/go-chi/chi"
1213
"github.com/go-chi/cors"
@@ -89,8 +90,9 @@ func main() {
8990

9091
router.Mount("/v1", v1Router)
9192
srv := &http.Server{
92-
Addr: ":" + port,
93-
Handler: router,
93+
Addr: ":" + port,
94+
Handler: router,
95+
ReadHeaderTimeout: 4 * time.Second,
9496
}
9597

9698
log.Printf("Serving on port: %s\n", port)

0 commit comments

Comments
 (0)