Fix linting (#4)

* Fix linting

Signed-off-by: Dan Webb <dan.webb@damacus.io>

* fix: replace deprecated LoggerWithConfig with RequestLoggerWithConfig

---------

Signed-off-by: Dan Webb <dan.webb@damacus.io>

Author: damacus

cmd/server/main.go

@@ -35,7 +35,14 @@ func main() {
 	dashboardHandler := handlers.NewDashboardHandler(minioFactory)
 
 	// Middleware
-	e.Use(middleware.Logger())
+	e.Use(middleware.RequestLoggerWithConfig(middleware.RequestLoggerConfig{
+		LogStatus: true,
+		LogURI:    true,
+		LogValuesFunc: func(c echo.Context, v middleware.RequestLoggerValues) error {
+			log.Printf("REQUEST: uri: %v, status: %v\n", v.URI, v.Status)
+			return nil
+		},
+	}))
 	e.Use(middleware.Recover())
 	// Apply auth middleware globally - it will skip public routes internally
 	e.Use(customMiddleware.AuthMiddleware(authService))

internal/services/auth_service_test.go

@@ -11,9 +11,6 @@ func TestNewAuthService_GeneratesKey(t *testing.T) {
 
 	svc := NewAuthService()
 
-	if svc == nil {
-		t.Fatal("expected non-nil AuthService")
-	}
 	if len(svc.encryptionKey) != 32 {
 		t.Errorf("expected 32-byte key, got %d bytes", len(svc.encryptionKey))
 	}
@@ -26,9 +23,6 @@ func TestNewAuthService_UsesEnvKey(t *testing.T) {
 
 	svc := NewAuthService()
 
-	if svc == nil {
-		t.Fatal("expected non-nil AuthService")
-	}
 	if string(svc.encryptionKey) != testKey {
 		t.Errorf("expected key %q, got %q", testKey, string(svc.encryptionKey))
 	}
@@ -40,9 +34,6 @@ func TestNewAuthService_IgnoresShortEnvKey(t *testing.T) {
 
 	svc := NewAuthService()
 
-	if svc == nil {
-		t.Fatal("expected non-nil AuthService")
-	}
 	// Should generate a new key since env key is too short
 	if len(svc.encryptionKey) != 32 {
 		t.Errorf("expected 32-byte generated key, got %d bytes", len(svc.encryptionKey))