From beeb01d0a08d2ccbe8274ac2bded0e2be028e087 Mon Sep 17 00:00:00 2001
From: Saloni <sa.choudhary@f5.com>
Date: Thu, 5 Sep 2024 13:16:56 -0600
Subject: [PATCH] updates based on reviews

---
 apis/v1alpha1/nginxproxy_types.go                   | 5 ++---
 internal/mode/static/state/graph/nginxproxy.go      | 2 +-
 internal/mode/static/state/graph/nginxproxy_test.go | 2 +-
 site/content/reference/api.md                       | 2 +-
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/apis/v1alpha1/nginxproxy_types.go b/apis/v1alpha1/nginxproxy_types.go
index 431adeeab5..acb42e4f5e 100644
--- a/apis/v1alpha1/nginxproxy_types.go
+++ b/apis/v1alpha1/nginxproxy_types.go
@@ -155,10 +155,9 @@ type RewriteClientIP struct {
 	// Sets NGINX directive set_real_ip_from: https://nginx.org/en/docs/http/ngx_http_realip_module.html#set_real_ip_from
 	// This field is required if mode is set.
 	// +kubebuilder:validation:MaxItems=16
-	// 	+listType=map
+	// +listType=map
 	// +listMapKey=type
 	//
-	//
 	// +optional
 	TrustedAddresses []Address `json:"trustedAddresses,omitempty"`
 }
@@ -201,6 +200,6 @@ type AddressType string
 
 const (
 	// AddressTypeCIDR specifies that the address is a CIDR block.
-	// kubebuilder:validation:Pattern=`^[\.a-zA-Z0-9::]*(\/([0-9]?[0-9]?[0-8]))$`
+	// kubebuilder:validation:Pattern=`^[\.a-zA-Z0-9:]*(\/([0-9]?[0-9]?[0-9]))$`
 	AddressTypeCIDR AddressType = "cidr"
 )
diff --git a/internal/mode/static/state/graph/nginxproxy.go b/internal/mode/static/state/graph/nginxproxy.go
index 2c1cd3ffb9..cf6dc70990 100644
--- a/internal/mode/static/state/graph/nginxproxy.go
+++ b/internal/mode/static/state/graph/nginxproxy.go
@@ -186,7 +186,7 @@ func validateRewriteClientIP(npCfg *ngfAPI.NginxProxy) field.ErrorList {
 			default:
 				allErrs = append(
 					allErrs,
-					field.NotSupported(trustedAddressesPath.Child(addr.Value),
+					field.NotSupported(trustedAddressesPath.Child("type"),
 						addr.Type,
 						[]string{string(ngfAPI.AddressTypeCIDR)},
 					),
diff --git a/internal/mode/static/state/graph/nginxproxy_test.go b/internal/mode/static/state/graph/nginxproxy_test.go
index 8603598f5c..8c9f236237 100644
--- a/internal/mode/static/state/graph/nginxproxy_test.go
+++ b/internal/mode/static/state/graph/nginxproxy_test.go
@@ -535,7 +535,7 @@ func TestValidateRewriteClientIP(t *testing.T) {
 				},
 			},
 			expectErrCount: 1,
-			errorString: "spec.rewriteClientIP.trustedAddresses.2001:db8::/129: " +
+			errorString: "spec.rewriteClientIP.trustedAddresses.type: " +
 				"Unsupported value: \"invalid\": supported values: \"cidr\"",
 		},
 	}
diff --git a/site/content/reference/api.md b/site/content/reference/api.md
index 702e758b2e..d5b191193b 100644
--- a/site/content/reference/api.md
+++ b/site/content/reference/api.md
@@ -533,7 +533,7 @@ string
 </thead>
 <tbody><tr><td><p>&#34;cidr&#34;</p></td>
 <td><p>AddressTypeCIDR specifies that the address is a CIDR block.
-kubebuilder:validation:Pattern=<code>^[\.a-zA-Z0-9::]*(\/([0-9]?[0-9]?[0-8]))$</code></p>
+kubebuilder:validation:Pattern=<code>^[\.a-zA-Z0-9:]*(\/([0-9]?[0-9]?[0-9]))$</code></p>
 </td>
 </tr></tbody>
 </table>