Merge branch 'develop'

.drone.yml

@@ -7,7 +7,7 @@ steps:
   image: golang:latest
   commands:
   - go install honnef.co/go/tools/cmd/staticcheck@latest
-  - cd src && staticcheck ./src/internal/...
+  - cd src && staticcheck ./...
   volumes:
   - name: gopath
     path: /go
@@ -16,7 +16,7 @@ steps:
   image: golang:latest
   commands:
   - go install golang.org/x/lint/golint@latest
-  - golint ./src/internal/...
+  - golint ./src/...
   volumes:
   - name: gopath
     path: /go
@@ -24,7 +24,7 @@ steps:
 - name: analyze
   image: golang:latest
   commands:
-  - go vet ./src/internal/...
+  - cd src && go vet ./...
   volumes:
   - name: gopath
     path: /go

src/cmd/main.go

@@ -14,9 +14,9 @@ import (
 )
 
 const (
-	defHttpIp   = "0.0.0.0"
-	defHttpPort = "8080"
-	defDbUrl    = "postgres://postgres:12345678@127.0.0.1:5432/egommerce"
+	defHTTPIP   = "0.0.0.0"
+	defHTTPPort = "8080"
+	defDbURL    = "postgres://postgres:12345678@127.0.0.1:5432/egommerce"
 )
 
 func main() {
@@ -24,15 +24,15 @@ func main() {
 		app.Panicf("Error loading .env file")
 	}
 
-	httpAddr := net.JoinHostPort(config.GetEnv("SERVER_IP", defHttpIp), defHttpPort)
-	dbConnStr := config.GetEnv("DATABASE_URL", defDbUrl)
+	httpAddr := net.JoinHostPort(config.GetEnv("SERVER_IP", defHTTPIP), defHTTPPort)
+	dbConnStr := config.GetEnv("DATABASE_URL", defDbURL)
 
 	dbc, err := database.Connect(dbConnStr)
 	if err != nil {
 		app.Panicf("Unable to connect to database: %v\n", err)
 	}
 
-	env := &handler.Env{httpAddr, dbc}
+	env := &handler.Env{Addr: httpAddr, DB: dbc}
 	srv := app.NewServer(env)
 
 	go srv.Start()

src/internal/app/handler/handler.go

@@ -3,7 +3,7 @@ package handler
 import (
 	"bytes"
 	"encoding/json"
-	"io/ioutil"
+	"io"
 	"log"
 	"net/http"
 
@@ -55,9 +55,9 @@ func (h *Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 }
 
 func decodeRequestData(r *http.Request, v interface{}) error {
-	buf, _ := ioutil.ReadAll(r.Body)
-	rdr := ioutil.NopCloser(bytes.NewReader(buf))
-	r.Body = ioutil.NopCloser(bytes.NewReader(buf))
+	buf, _ := io.ReadAll(r.Body)
+	rdr := io.NopCloser(bytes.NewReader(buf))
+	r.Body = io.NopCloser(bytes.NewReader(buf))
 
 	json.NewDecoder(rdr).Decode(&v)
 

src/internal/app/router.go

@@ -13,7 +13,7 @@ func SetupRouter(env *handler.Env) *mux.Router {
 	r.MethodNotAllowedHandler = &handler.MethodNotAllowedHandler{}
 
 	r.Use(PrepareHeadersMiddleware)
-	r.Use(ValidateJsonBodyMiddleware) // probably not needed
+	r.Use(ValidateJSONBodyMiddleware) // probably not needed
 	r.Use(LoggingMiddleware)
 
 	r.Handle("/health", handler.Init(env, handler.HealthCheckHandler)).Methods(http.MethodGet)

src/internal/app/server.go

@@ -3,7 +3,7 @@ package app
 import (
 	"bytes"
 	"encoding/json"
-	"io/ioutil"
+	"io"
 	"log"
 	"net"
 	"net/http"
@@ -61,10 +61,10 @@ func PrepareHeadersMiddleware(next http.Handler) http.Handler {
 	})
 }
 
-func ValidateJsonBodyMiddleware(next http.Handler) http.Handler {
+func ValidateJSONBodyMiddleware(next http.Handler) http.Handler {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		buf, _ := ioutil.ReadAll(r.Body)
-		r.Body = ioutil.NopCloser(bytes.NewReader(buf)) // rollack *Request to original state
+		buf, _ := io.ReadAll(r.Body)
+		r.Body = io.NopCloser(bytes.NewReader(buf)) // rollack *Request to original state
 
 		if len(buf) > 0 && !json.Valid(buf) {
 			w.WriteHeader(http.StatusBadRequest)