Merge pull request #3 from bloodsweatncode/addstats

Introduced Linting
Introduced staticcheck

Author: bloodsweatncode

.github/workflows/ci.yml

@@ -21,6 +21,12 @@ jobs:
       - name: Run Tests
         run: go test --cover ./...
 
+      - name: Install gosec
+        run: go install github.com/securego/gosec/v2/cmd/gosec@latest
+
+      - name: Run gosec
+        run: gosec ./...
+
   style:
     name: Style
     runs-on: ubuntu-latest
@@ -34,5 +40,11 @@ jobs:
         with:
           go-version: "1.23.0"
 
+      - name: Install staticcheck
+        run: go install honnef.co/go/tools/cmd/staticcheck@latest
+
       - name: Check Format
         run: test -z $(go fmt ./...)
+
+      - name: Linting
+        run: staticcheck ./...

json.go

@@ -30,5 +30,8 @@ func respondWithJSON(w http.ResponseWriter, code int, payload interface{}) {
 		return
 	}
 	w.WriteHeader(code)
-	w.Write(dat)
+	_, writeErr := w.Write(dat)
+	if writeErr != nil {
+		log.Printf("Error writing data: %s", err)
+	}
 }

main.go

@@ -7,6 +7,7 @@ import (
 	"log"
 	"net/http"
 	"os"
+	"time"
 
 	"github.com/go-chi/chi"
 	"github.com/go-chi/cors"
@@ -89,8 +90,10 @@ func main() {
 
 	router.Mount("/v1", v1Router)
 	srv := &http.Server{
-		Addr:    ":" + port,
-		Handler: router,
+		Addr:         ":" + port,
+		ReadTimeout:  5 * time.Second,
+		WriteTimeout: 10 * time.Second,
+		Handler:      router,
 	}
 
 	log.Printf("Serving on port: %s\n", port)