Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

removeUnusedFilters doesn't remove CORS and WAF filters #6042

Closed
guydc opened this issue Mar 8, 2022 · 2 comments
Closed

removeUnusedFilters doesn't remove CORS and WAF filters #6042

guydc opened this issue Mar 8, 2022 · 2 comments
Assignees
@sam-heilbron
Labels
Type: Bug Something isn't working
Milestone
Q2-2

Comments

@guydc
Copy link

guydc commented Mar 8, 2022

Gloo Edge Version

1.10.x (latest stable)

Kubernetes Version

1.22.x

Describe the bug

Both cors and modsecurity are added to the chain despite removeUnusedFilters: true and not having any explicit WAF or CORS configuration made.

Steps to reproduce the bug

deploy gloo edge, configure settings, create simple VS, observe generated filter chain in evnoy config dump.

Expected Behavior

cors and modsecurity filters should not be injected into HTTP filter chain.

Additional Context

No response
@guydc guydc added the Type: Bug Something isn't working label Mar 8, 2022
@chrisgaun
Copy link

Related to this #5651

@chrisgaun chrisgaun modified the milestone: Q2-2 Apr 12, 2022
@chrisgaun chrisgaun added this to the Q2-2 milestone May 3, 2022
@solo-changelog-bot solo-changelog-bot bot mentioned this issue May 4, 2022
Merged
8 tasks
@sam-heilbron sam-heilbron self-assigned this May 4, 2022
@solo-changelog-bot solo-changelog-bot bot mentioned this issue May 4, 2022
Merged
10 tasks
@sam-heilbron
Copy link
Contributor

This is merged and released in Gloo EE 1.12 and 1.11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sam-heilbron sam-heilbron

Labels
Type: Bug Something isn't working
Projects
None yet
Milestone
Q2-2
Development

No branches or pull requests
3 participants
@sam-heilbron @chrisgaun @guydc