Skip to content

Commit e11ed2f

Browse files
committed
Remove maximum snippet size validation from SnippetsPolicy
1 parent f47f54e commit e11ed2f

File tree

2 files changed

+0
-21
lines changed

2 files changed

+0
-21
lines changed

internal/controller/nginx/config/policies/snippetspolicy/validator.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,6 @@ func (v *Validator) Conflicts(polA, polB policies.Policy) bool {
7777
return false
7878
}
7979

80-
const maxSnippetSize = 2048 // 2KB
81-
8280
func validateSnippets(snippets []ngfAPI.Snippet) error {
8381
seenContexts := make(map[ngfAPI.NginxContext]struct{})
8482
for _, snippet := range snippets {
@@ -90,10 +88,6 @@ func validateSnippets(snippets []ngfAPI.Snippet) error {
9088
if snippet.Context == ngfAPI.NginxContextHTTPServerLocation {
9189
return fmt.Errorf("context %q is not supported in SnippetsPolicy", snippet.Context)
9290
}
93-
94-
if len(snippet.Value) > maxSnippetSize {
95-
return fmt.Errorf("snippet value for context %q exceeds maximum size of %d bytes", snippet.Context, maxSnippetSize)
96-
}
9791
}
9892
return nil
9993
}

internal/controller/nginx/config/policies/snippetspolicy/validator_test.go

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -120,21 +120,6 @@ func TestValidator_Validate(t *testing.T) {
120120
conditions.NewPolicyInvalid("context \"http.server.location\" is not supported in SnippetsPolicy"),
121121
},
122122
},
123-
{
124-
name: "snippet too large",
125-
policy: createModifiedPolicy(func(p *ngfAPI.SnippetsPolicy) *ngfAPI.SnippetsPolicy {
126-
p.Spec.Snippets = []ngfAPI.Snippet{
127-
{
128-
Context: ngfAPI.NginxContextMain,
129-
Value: string(make([]byte, 2049)),
130-
},
131-
}
132-
return p
133-
}),
134-
expConditions: []conditions.Condition{
135-
conditions.NewPolicyInvalid("snippet value for context \"main\" exceeds maximum size of 2048 bytes"),
136-
},
137-
},
138123
}
139124

140125
v := snippetspolicy.NewValidator()

0 commit comments

Comments
 (0)