fix: lint changes

Author: blockchainluffy

rolling-shutter/keyper/kproapi/middleware.go

@@ -7,32 +7,32 @@ import (
 	"github.com/getkin/kin-openapi/openapi3"
 )
 
-// isReadOnlyEndpoint checks if an endpoint is marked as read-only in the OpenAPI spec
+// isReadOnlyEndpoint checks if an endpoint is marked as read-only in the OpenAPI spec.
 func isReadOnlyEndpoint(operation *openapi3.Operation) bool {
-	// Try to get the value directly from the map first
+	// Try to get the value directly from the map first.
 	if val, exists := operation.Extensions["x-read-only"]; exists {
-		// Handle json.RawMessage case
+		// Handle json.RawMessage case.
 		if rawMsg, ok := val.(json.RawMessage); ok {
 			return string(rawMsg) == "true"
 		}
 
-		// Handle direct boolean case
+		// Handle direct boolean case.
 		if boolVal, ok := val.(bool); ok {
 			return boolVal
 		}
 	}
 	return false
 }
 
-// shouldEnableEndpoint determines if an endpoint should be accessible based on its type and configuration
+// shouldEnableEndpoint determines if an endpoint should be accessible based on its type and configuration.
 func shouldEnableEndpoint(operation *openapi3.Operation, enableWriteOperations bool) bool {
 	if isReadOnlyEndpoint(operation) {
 		return true
 	}
 	return enableWriteOperations
 }
 
-// findOperation looks up the OpenAPI operation for the given path and method
+// findOperation looks up the OpenAPI operation for the given path and method.
 func findOperation(spec *openapi3.T, path string, method string) *openapi3.Operation {
 	pathItem := spec.Paths.Find(path)
 	if pathItem == nil {
@@ -53,31 +53,31 @@ func findOperation(spec *openapi3.T, path string, method string) *openapi3.Opera
 	}
 }
 
-// ConfigMiddleware creates a middleware that controls endpoint access based on configuration
+// ConfigMiddleware creates a middleware that controls endpoint access based on configuration.
 func ConfigMiddleware(enableWriteOperations bool) MiddlewareFunc {
 	return ConfigMiddlewareWithSpec(enableWriteOperations, GetSwagger)
 }
 
-// ConfigMiddlewareWithSpec creates a middleware that controls endpoint access based on configuration
-// This accepts a function to get the spec, making it more testable
+// ConfigMiddlewareWithSpec creates a middleware that controls endpoint access based on configuration.
+// This accepts a function to get the spec, making it more testable.
 func ConfigMiddlewareWithSpec(enableWriteOperations bool, getSpec func() (*openapi3.T, error)) MiddlewareFunc {
 	return func(next http.Handler) http.Handler {
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-			// Load the OpenAPI specification
+			// Load the OpenAPI specification.
 			spec, err := getSpec()
 			if err != nil {
 				http.Error(w, "Internal server error", http.StatusInternalServerError)
 				return
 			}
 
-			// Find the operation for this request
+			// Find the operation for this request.
 			operation := findOperation(spec, r.URL.Path, r.Method)
 			if operation == nil {
 				http.Error(w, "Endpoint not found", http.StatusNotFound)
 				return
 			}
 
-			// Check if the endpoint should be accessible
+			// Check if the endpoint should be accessible.
 			if !shouldEnableEndpoint(operation, enableWriteOperations) {
 				http.Error(w, "Endpoint not enabled", http.StatusForbidden)
 				return

rolling-shutter/keyper/kproapi/middleware_test.go

@@ -201,7 +201,8 @@ func TestConfigMiddleware(t *testing.T) {
 		},
 	}
 
-	// Create a function that returns our test spec
+	// Create a function that returns our test spec.
+	//nolint:golint
 	getTestSpec := func() (*openapi3.T, error) {
 		return spec, nil
 	}
@@ -245,8 +246,8 @@ func TestConfigMiddleware(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			// Create a test handler that always returns 200 OK
-			nextHandler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+			// Create a test handler that always returns 200 OK.
+			nextHandler := http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 				w.WriteHeader(http.StatusOK)
 			})
 
@@ -255,7 +256,7 @@ func TestConfigMiddleware(t *testing.T) {
 			handler := middleware(nextHandler)
 
 			// Create a test request
-			req := httptest.NewRequest(tt.method, tt.path, nil)
+			req := httptest.NewRequest(tt.method, tt.path, http.NoBody)
 			w := httptest.NewRecorder()
 
 			// Serve the request