Enhance ingress path handling and annotation generation

Author: ziyang-lin-404

internal/controller/scalityuicomponentexposer/controller_test.go

@@ -396,4 +396,53 @@ var _ = Describe("ScalityUIComponentExposer Controller", func() {
 			_ = k8sClient.Delete(ctx, uiWithNetworks)
 		})
 	})
+
+	Context("When testing path handling", func() {
+		It("should preserve original path format without forcing trailing slashes", func() {
+			networks := &uiv1alpha1.UINetworks{
+				Host: "test.example.com",
+			}
+
+			By("Testing path without trailing slash - should remain as-is")
+			rules := getIngressRules(networks, "/test-component")
+			Expect(rules).To(HaveLen(1))
+			Expect(rules[0].Host).To(Equal("test.example.com"))
+			Expect(rules[0].Path).To(Equal("/test-component"))
+
+			By("Testing path with trailing slash - should remain as-is")
+			rules = getIngressRules(networks, "/test-component/")
+			Expect(rules).To(HaveLen(1))
+			Expect(rules[0].Path).To(Equal("/test-component/"))
+
+			By("Testing empty path - should default to root")
+			rules = getIngressRules(networks, "")
+			Expect(rules).To(HaveLen(1))
+			Expect(rules[0].Path).To(Equal("/"))
+
+			By("Testing root path - should remain as-is")
+			rules = getIngressRules(networks, "/")
+			Expect(rules).To(HaveLen(1))
+			Expect(rules[0].Path).To(Equal("/"))
+		})
+
+		It("should generate flexible nginx rewrite rules for runtime configuration", func() {
+			networks := &uiv1alpha1.UINetworks{
+				Host: "test.example.com",
+			}
+
+			By("Testing annotation generation for path without trailing slash")
+			annotations := getIngressAnnotations(networks, "/test-app", "test-exposer")
+			Expect(annotations).To(HaveKey("nginx.ingress.kubernetes.io/configuration-snippet"))
+
+			snippet := annotations["nginx.ingress.kubernetes.io/configuration-snippet"]
+			// Should match both /test-app/.well-known/... and /test-app//.well-known/...
+			Expect(snippet).To(ContainSubstring("^/test-app/?/?"))
+
+			By("Testing annotation generation for path with trailing slash")
+			annotations = getIngressAnnotations(networks, "/test-app/", "test-exposer")
+			snippet = annotations["nginx.ingress.kubernetes.io/configuration-snippet"]
+			// Should normalize to /test-app and still be flexible
+			Expect(snippet).To(ContainSubstring("^/test-app/?/?"))
+		})
+	})
 })

internal/controller/scalityuicomponentexposer/ingress.go

@@ -2,6 +2,7 @@ package scalityuicomponentexposer
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/scality/reconciler-framework/reconciler"
 	"github.com/scality/reconciler-framework/resources"
@@ -98,10 +99,17 @@ func getIngressClassName(networks *uiv1alpha1.UINetworks) string {
 
 // getIngressRules returns the ingress rules based on networks config
 func getIngressRules(networks *uiv1alpha1.UINetworks, path string) []resources.IngressHostPath {
+	// Use the path as-is, without forcing trailing slash
+	// Default to "/" if path is empty
+	ingressPath := path
+	if ingressPath == "" {
+		ingressPath = "/"
+	}
+
 	rules := []resources.IngressHostPath{
 		{
 			Host: networks.Host,
-			Path: path + "/",
+			Path: ingressPath,
 		},
 	}
 	return rules
@@ -123,11 +131,13 @@ func getIngressAnnotations(networks *uiv1alpha1.UINetworks, path string, exposer
 
 	// Add rewrite annotation for exposer runtime configuration path
 	// Use configuration-snippet for conditional rewriting
+	// Handle both paths with and without trailing slashes
+	normalizedPath := strings.TrimSuffix(path, "/")
 	configSnippet := fmt.Sprintf(`
-if ($request_uri ~ "^%s/\\.well-known/runtime-app-configuration(\\?.*)?$") {
+if ($request_uri ~ "^%s/?/?\\.well-known/runtime-app-configuration(\\?.*)?$") {
     rewrite ^.*$ /.well-known/%s/%s break;
 }
-`, path, configsSubdirectory, exposerName)
+`, normalizedPath, configsSubdirectory, exposerName)
 	annotations["nginx.ingress.kubernetes.io/configuration-snippet"] = configSnippet
 
 	return annotations