Reduce cognitive complexity of router builder code

application/service/builder.go

@@ -55,15 +55,11 @@ func getHandlerForRoute(path string, route config.Route, log zerolog.Logger) fun
 				w.Header().Set(headerName, headerValue)
 			}
 		}
-		if status == nil {
-			status = utils.PointerTo(http.StatusOK)
-		}
-		if contentType == nil {
-			contentType = utils.PointerTo("text/plain")
-		}
-		if content == nil {
-			content = new(string)
-		}
+
+		status = utils.GetOptionalOrFallback(status, utils.PointerTo(http.StatusOK))
+		contentType = utils.GetOptionalOrFallback(contentType, utils.PointerTo("text/plain"))
+		content = utils.GetOptionalOrFallback(content, new(string))
+
 		if strings.HasPrefix(strings.TrimSpace(*content), "base64,") {
 			var err error
 			finalContent, err = base64.StdEncoding.DecodeString(strings.TrimPrefix(strings.TrimSpace(*content), "base64,"))

application/utils/optional.go

@@ -0,0 +1,9 @@
+package utils
+
+// GetOptionalOrFallback returns pointer to value if value is not empty or fallback pointer otherwise
+func GetOptionalOrFallback[T any](value *T, fallback *T) *T {
+	if value == nil {
+		return fallback
+	}
+	return value
+}

application/utils/optional_test.go

@@ -0,0 +1,34 @@
+package utils
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestGetOptionalOrFallback(t *testing.T) {
+	someValue, someFallback := PointerTo("someValue"), PointerTo("someFallback")
+	tests := map[string]struct {
+		value         *string
+		fallback      *string
+		expectedValue *string
+	}{
+		"value is not nil": {
+			value:         someValue,
+			fallback:      someFallback,
+			expectedValue: someValue,
+		},
+		"value is nil": {
+			value:         nil,
+			fallback:      someFallback,
+			expectedValue: someFallback,
+		},
+	}
+
+	for name, tc := range tests {
+		name, tc := name, tc
+		t.Run(name, func(t *testing.T) {
+			require.Equal(t, tc.expectedValue, GetOptionalOrFallback(tc.value, tc.fallback))
+		})
+	}
+}