Skip to content

Conversation

@eerhardt
Copy link
Member

These policies are required by Microsoft to prevent DDOS attacks on the site.

These policies are required by Microsoft to prevent DDOS attacks on the site.
@eerhardt eerhardt requested a review from IEvangelist as a code owner January 30, 2026 20:47
Copilot AI review requested due to automatic review settings January 30, 2026 20:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an Azure Front Door WAF policy and associates it with the Front Door endpoint to meet DDoS/WAF compliance requirements.

Changes:

  • Introduces a configurable rateLimitThreshold parameter for WAF rate limiting.
  • Adds a Front Door WAF Policy with a global rate-limit custom rule and a managed Bot Manager ruleset.
  • Associates the WAF policy to the Front Door endpoint via a Front Door security policy.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@IEvangelist IEvangelist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM, but some of the copilot comments are interesting. Might be worth considering...

@eerhardt
Copy link
Member Author

I'm getting an error trying to deploy this:

WebApplicationFirewallPolicy validation failed. More information "Policy ArmResourceId has incorrect formatting". Click here for details

trying to get copilot to fix it.

@IEvangelist IEvangelist merged commit ed14c0c into main Jan 31, 2026
6 checks passed
@IEvangelist IEvangelist deleted the eerhardt/AddWAF branch January 31, 2026 16:16
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.

3 participants