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.

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

Waf labels #1639

Merged
merged 18 commits into from Apr 18, 2024
Merged

Waf labels #1639

merged 18 commits into from Apr 18, 2024

Conversation

ChrisMcGowan
Copy link
Contributor

Changes proposed in this pull request:

  • Update logic to allow more then one host for waf label and waf log drop
  • update logic on app listener rule to route this traffic to separate router group

Notes

This allows internal CUPs apps from customers to bypass WAF CloudWatch logging to save on logging costs as well as shift this workload off the main routers

Thanks to @markdboyd for the assist.

security considerations

n/a

@ChrisMcGowan ChrisMcGowan requested a review from a team April 18, 2024 16:35
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov>
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov>
ChrisMcGowan and others added 3 commits April 18, 2024 12:37
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov>
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov>
@ChrisMcGowan ChrisMcGowan requested a review from a team April 18, 2024 16:39
Co-authored-by: Mark Boyd <mark.boyd@gsa.gov>
@ChrisMcGowan ChrisMcGowan merged commit d07dc2b into main Apr 18, 2024
1 of 2 checks passed
@ChrisMcGowan ChrisMcGowan deleted the waf_labels branch April 18, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants