Refactor listener naming to standardize HTTP and HTTP2 prefixes

Author: raniellyferreira

pkg/cloudprovider/providers/oci/load_balancer_spec.go

@@ -1163,7 +1163,7 @@ func getListenersOciLoadBalancer(svc *v1.Service, sslCfg *SSLConfig) (map[string
 			// At that point LB only supports HTTP and TCP
 			defaultIdleTimeoutPerProtocol := map[string]int64{
 				"HTTP":  lbConnectionIdleTimeoutHTTP,
-				"HTTP2": lbConnectionIdleTimeoutHTTP, // HTTP2 uses same timeout as HTTP
+				"HTTP2": lbConnectionIdleTimeoutHTTP, // HTTP/2 uses same timeout as HTTP
 				"TCP":   lbConnectionIdleTimeoutTCP,
 			}
 			actualConnectionIdleTimeout = common.Int64(defaultIdleTimeoutPerProtocol[strings.ToUpper(listenerProtocol)])

pkg/cloudprovider/providers/oci/load_balancer_spec_test.go

@@ -8964,8 +8964,8 @@ func Test_getListeners(t *testing.T) {
 			listenerBackendIpVersion: []string{IPv4},
 			sslConfig:                nil,
 			want: map[string]client.GenericListener{
-				"HTTP2-443": {
-					Name:                  common.String("HTTP2-443"),
+				"HTTP-443": {
+					Name:                  common.String("HTTP-443"),
 					Port:                  common.Int(443),
 					Protocol:              common.String("HTTP2"),
 					DefaultBackendSetName: common.String("TCP-443"),

pkg/cloudprovider/providers/oci/load_balancer_util.go

@@ -679,6 +679,10 @@ func getSanitizedName(name string) string {
 }
 
 func getListenerName(protocol string, port int) string {
+	// For HTTP and HTTP/2 protocols, always use "HTTP" prefix
+	if strings.EqualFold(protocol, "HTTP") || strings.EqualFold(protocol, "HTTP2") {
+		return fmt.Sprintf("HTTP-%d", port)
+	}
 	return fmt.Sprintf("%s-%d", protocol, port)
 }