backout APIKey skip_metadata_validation API change (#10188)

changelog/v1.15.34/validate-apikey-data.yaml

@@ -0,0 +1,6 @@
+changelog:
+  - type: NON_USER_FACING
+    issueLink: https://github.com/solo-io/ext-auth-service/issues/762
+    resolvesIssue: false
+    description: >-
+      Revert updating extauth APIKey config to add 'skipMetadataValidation', as we will not be backporting to this version

install/helm/gloo/crds/enterprise.gloo.solo.io_v1_AuthConfig.yaml

@@ -158,8 +158,6 @@ spec:
                           additionalProperties:
                             type: string
                           type: object
-                        skipMetadataValidation:
-                          type: boolean
                       type: object
                     basicAuth:
                       properties:

projects/gloo/api/v1/enterprise/options/extauth/v1/extauth.proto

@@ -1023,12 +1023,8 @@ message ApiKeyAuth {
         // to the request.
         bool required = 2;
     }
-
-    // API key metadata may contain data is is invalid for a header, such as a newline. By default, this data will be validated
-    // in the data plane and mitigated in a way that provides a consistent experience for the user and visibility for the operator.
-    // This validation comes with a performance cost, and can be disabled by setting this field to `true`.
-    bool skip_metadata_validation = 8;
 }
+
 message K8sSecretApiKeyStorage {
     // Identify all valid API key secrets that match the provided label selector.<br/>
     // API key secrets must be in one of the watch namespaces for gloo to locate them.
@@ -1852,11 +1848,6 @@ message ExtAuthConfig {
             K8sSecretApiKeyStorage k8s_secret_apikey_storage = 4;
             AerospikeApiKeyStorage aerospike_apikey_storage = 5;
         }
-
-        // API key metadata may contain data is is invalid for a header, such as a newline. By default, this data will be validated
-        // in the data plane and mitigated in a way that provides a consistent experience for the user and visibility for the operator.
-        // This validation comes with a performance cost, and can be disabled by setting this field to `true`.
-        bool skip_metadata_validation = 6;
     }
 
     message OpaAuthConfig {