Merged in feature/lint (pull request #87)

Basic Lint Checks

* basic

internal/serviceconfig/observability/config_test.go

@@ -3,9 +3,10 @@ package observability_test
 import (
 	"context"
 	"fmt"
-	"queryorchestration/internal/serviceconfig/observability"
 	"testing"
 
+	"queryorchestration/internal/serviceconfig/observability"
+
 	"github.com/stretchr/testify/assert"
 )
 

internal/serviceconfig/observability/prometheus/prometheus.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"net/http"
 	"sync"
+	"time"
 
 	"github.com/prometheus/client_golang/prometheus"
 	"github.com/prometheus/client_golang/prometheus/promhttp"
@@ -121,8 +122,9 @@ func New(ctx context.Context, config MetricsConfig) (*Metrics, error) {
 		mux.Handle("/metrics", promhttp.Handler())
 
 		m.server = &http.Server{
-			Addr:    fmt.Sprintf(":%d", config.Port),
-			Handler: mux,
+			Addr:              fmt.Sprintf(":%d", config.Port),
+			Handler:           mux,
+			ReadHeaderTimeout: time.Minute,
 		}
 
 		go func() {

internal/serviceconfig/observability/prometheus/prometheus_test.go

@@ -27,7 +27,12 @@ func GetFreePortForTesting() (int, error) {
 	defer l.Close()
 
 	// Get the port that was assigned by the OS
-	return l.Addr().(*net.TCPAddr).Port, nil
+	addr, ok := l.Addr().(*net.TCPAddr)
+	if !ok {
+		return 0, fmt.Errorf("failed to parse TCP address")
+	}
+
+	return addr.Port, nil
 }
 
 func TestMetricsIntegration(t *testing.T) {