Skip to content

Conversation

@cgoenka1
Copy link
Contributor

With the Customer Provided IP option, DDoS should be enabled for secure hub.

@prmerger-automator
Copy link
Contributor

@cgoenka1 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 84b98fd:

✅ Validation status: passed

File Status Preview URL Details
articles/firewall-manager/vhubs-and-vnets.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner
Copy link
Contributor

@duongau - Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Aug 20, 2025
@v-dirichards
Copy link
Contributor

@duongau Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

@v-ccolin
Copy link
Contributor

v-ccolin commented Oct 3, 2025

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards v-dirichards requested a review from Copilot October 9, 2025 21:54
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

Updates documentation to reflect that Azure DDoS Protection is now supported for secure hubs when using Customer Provided IP option.

  • Modified DDoS Protection support documentation for secure hubs
  • Added clarification that DDoS support requires Customer Provided IP option

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 420b4d3:

✅ Validation status: passed

File Status Preview URL Details
articles/firewall-manager/vhubs-and-vnets.md ✅Succeeded

For more details, please refer to the build report.

@v-ccolin
Copy link
Contributor

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

|**Web Application Firewall on Application Gateway** |Supported in Virtual Network|Currently supported in spoke network|
|**Network Virtual Appliance**|Supported in Virtual Network|Currently supported in spoke network|
|**Azure DDoS Protection support**|Yes|No|
|**Azure DDoS Protection support**|Yes|Yes (With Customer Provided IP)|
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
|**Azure DDoS Protection support**|Yes|Yes (With Customer Provided IP)|
|**Azure DDoS Protection support**|Yes|Yes ([With customer provided public IP](../firewall/secured-hub-customer-public-ip.md))|

@duongau
Copy link
Contributor

duongau commented Nov 16, 2025

Hi @v-dirichards and @v-ccolin, please update the PR with my suggested fix. We can go merge after that. #sign-off

Updated the description for Azure DDoS Protection support to include details about customer-provided public IP address.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 56666e0:

✅ Validation status: passed

File Status Preview URL Details
articles/firewall-manager/vhubs-and-vnets.md ✅Succeeded

For more details, please refer to the build report.

@ktoliver
Copy link
Contributor

#sign-off

@prmerger-automator
Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @duongau

@ktoliver
Copy link
Contributor

Due to the upcoming conference, all merging and publishing is on hold until the morning of November 18, 2025.

This PR is ready for merge on November 18, 2025.

#label: "event-triaged-to-10AM"
#label: "missed-content-complete"
#label: "ready-to-merge" 

@v-dirichards v-dirichards merged commit 4ae6ef8 into MicrosoftDocs:main Nov 18, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants