api: ACL API design

Signed-off-by: Jesse Haka <haka.jesse@gmail.com>

api/v1alpha1/acl_types.go

@@ -0,0 +1,24 @@
+// Copyright Envoy Gateway Authors
+// SPDX-License-Identifier: Apache-2.0
+// The full text of the Apache license is available in the LICENSE file at
+// the root of the repo.
+
+package v1alpha1
+
+// +kubebuilder:validation:XValidation:rule="(has(self.allow) || has(self.deny))",message="one of allow or deny must be specified"
+//
+// ACL defines the IP deny/allow configuration.
+type ACL struct {
+	// Allow specifies the list of IPBlocks that are allowed to access the service.
+	// Other cidrs are denied.
+	Allow []IPBlock `json:"allow,omitempty"`
+	// Deny specifies the list of IPBlocks that are denied to access the service.
+	Deny []IPBlock `json:"deny,omitempty"`
+}
+
+// IPBlock defines policy on a particular IPBlock.
+type IPBlock struct {
+	// cidr is a string representing the IPBlock
+	// Valid examples are "192.168.1.0/24" or "2001:db8::/64"
+	CIDR string `json:"cidr,omitempty"`
+}

api/v1alpha1/securitypolicy_types.go

@@ -70,6 +70,11 @@ type SecurityPolicySpec struct {
 	//
 	// +optional
 	ExtAuth *ExtAuth `json:"extAuth,omitempty"`
+
+	// ACL defines the IP deny/allow configuration.
+	//
+	// +optional
+	ACL *ACL `json:"acl,omitempty"`
 }
 
 // SecurityPolicyStatus defines the state of SecurityPolicy

api/v1alpha1/validation/envoyproxy_validate_test.go

@@ -8,6 +8,7 @@ package validation
 import (
 	// Register embed
 	_ "embed"
+	"fmt"
 	"reflect"
 	"testing"
 
@@ -587,3 +588,60 @@ func TestGetEnvoyProxyComponentLevelArgs(t *testing.T) {
 		})
 	}
 }
+
+func TestValidateACL(t *testing.T) {
+	type testCase struct {
+		name      string
+		cidr      string
+		expectErr bool
+	}
+	testCases := []testCase{
+		{
+			name:      "nil acl",
+			cidr:      "",
+			expectErr: false,
+		},
+		{
+			name:      "valid ipv4 cidr",
+			cidr:      "192.168.1.0/24",
+			expectErr: false,
+		},
+		{
+			name:      "valid ipv6 cidr",
+			cidr:      "2001:db8::/64",
+			expectErr: false,
+		},
+		{
+			name:      "invalid ipv4 cidr",
+			cidr:      "1x0/foo",
+			expectErr: true,
+		},
+		{
+			name:      "valid ipv6 cidr",
+			cidr:      "invalid/cidr",
+			expectErr: true,
+		},
+	}
+
+	for _, tc := range testCases {
+		for _, block := range []string{"allow", "deny"} {
+			t.Run(fmt.Sprintf("%s/%s", tc.name, block), func(t *testing.T) {
+				var acl *egv1a1.ACL
+				if tc.cidr != "" {
+					acl = &egv1a1.ACL{}
+					if block == "allow" {
+						acl.Allow = []egv1a1.IPBlock{{CIDR: tc.cidr}}
+					} else if block == "deny" {
+						acl.Deny = []egv1a1.IPBlock{{CIDR: tc.cidr}}
+					}
+				}
+				err := ValidateACL(acl)
+				if tc.expectErr {
+					require.Error(t, err)
+				} else {
+					require.NoError(t, err)
+				}
+			})
+		}
+	}
+}

api/v1alpha1/validation/securitypolicy_validate.go

@@ -8,6 +8,7 @@ package validation
 import (
 	"errors"
 	"fmt"
+	"net"
 	"net/mail"
 	"net/url"
 
@@ -56,6 +57,27 @@ func validateSecurityPolicySpec(spec *egv1a1.SecurityPolicySpec) error {
 		errs = append(errs, err)
 	}
 
+	if err := ValidateACL(spec.ACL); err != nil {
+		errs = append(errs, err)
+	}
+
+	return utilerrors.NewAggregate(errs)
+}
+
+// ValidateACL validates the provided ACL configuration.
+func ValidateACL(acl *egv1a1.ACL) error {
+	var errs []error
+	if acl == nil {
+		return nil
+	}
+
+	for _, ipBlock := range append(acl.Allow, acl.Deny...) {
+		_, _, err := net.ParseCIDR(ipBlock.CIDR)
+		if err != nil {
+			errs = append(errs, fmt.Errorf("invalid CIDR: %s", ipBlock.CIDR))
+		}
+	}
+
 	return utilerrors.NewAggregate(errs)
 }
 

charts/gateway-helm/crds/generated/gateway.envoyproxy.io_securitypolicies.yaml

@@ -44,6 +44,37 @@ spec:
           spec:
             description: Spec defines the desired state of SecurityPolicy.
             properties:
+              acl:
+                description: ACL defines the IP deny/allow configuration.
+                properties:
+                  allow:
+                    description: Allow specifies the list of IPBlocks that are allowed
+                      to access the service. Other cidrs are denied.
+                    items:
+                      description: IPBlock defines policy on a particular IPBlock.
+                      properties:
+                        cidr:
+                          description: cidr is a string representing the IPBlock Valid
+                            examples are "192.168.1.0/24" or "2001:db8::/64"
+                          type: string
+                      type: object
+                    type: array
+                  deny:
+                    description: Deny specifies the list of IPBlocks that are denied
+                      to access the service.
+                    items:
+                      description: IPBlock defines policy on a particular IPBlock.
+                      properties:
+                        cidr:
+                          description: cidr is a string representing the IPBlock Valid
+                            examples are "192.168.1.0/24" or "2001:db8::/64"
+                          type: string
+                      type: object
+                    type: array
+                type: object
+                x-kubernetes-validations:
+                - message: one of allow or deny must be specified
+                  rule: (has(self.allow) || has(self.deny))
               basicAuth:
                 description: BasicAuth defines the configuration for the HTTP Basic
                   Authentication.

site/content/en/latest/api/extension_types.md

@@ -27,6 +27,21 @@ API group.
 
 
 
+#### ACL
+
+
+
+ACL defines the IP deny/allow configuration.
+
+_Appears in:_
+- [SecurityPolicySpec](#securitypolicyspec)
+
+| Field | Type | Required | Description |
+| ---   | ---  | ---      | ---         |
+| `allow` | _[IPBlock](#ipblock) array_ |  true  | Allow specifies the list of IPBlocks that are allowed to access the service. Other cidrs are denied. |
+| `deny` | _[IPBlock](#ipblock) array_ |  true  | Deny specifies the list of IPBlocks that are denied to access the service. |
+
+
 #### ALPNProtocol
 
 _Underlying type:_ _string_
@@ -1229,6 +1244,20 @@ _Appears in:_
 | `passive` | _[PassiveHealthCheck](#passivehealthcheck)_ |  false  | Passive passive check configuration |
 
 
+#### IPBlock
+
+
+
+IPBlock defines policy on a particular IPBlock.
+
+_Appears in:_
+- [ACL](#acl)
+
+| Field | Type | Required | Description |
+| ---   | ---  | ---      | ---         |
+| `cidr` | _string_ |  true  | cidr is a string representing the IPBlock Valid examples are "192.168.1.0/24" or "2001:db8::/64" |
+
+
 #### InfrastructureProviderType
 
 _Underlying type:_ _string_
@@ -2154,6 +2183,7 @@ _Appears in:_
 | `jwt` | _[JWT](#jwt)_ |  false  | JWT defines the configuration for JSON Web Token (JWT) authentication. |
 | `oidc` | _[OIDC](#oidc)_ |  false  | OIDC defines the configuration for the OpenID Connect (OIDC) authentication. |
 | `extAuth` | _[ExtAuth](#extauth)_ |  false  | ExtAuth defines the configuration for External Authorization. |
+| `acl` | _[ACL](#acl)_ |  false  | ACL defines the IP deny/allow configuration. |