Skip to content

Conversation

cpressland
Copy link
Contributor

This PR updates the Azure Firewall DNS Settings documentation to clarify that while Azure Firewall can resolve Azure Private DNS zones for its own operations, the DNS Proxy feature does not return these records to downstream clients unless the configured upstream DNS server can also resolve them. This helps prevent confusion for users expecting Private DNS zones to be accessible via the DNS Proxy without additional configuration.

@cpressland
Copy link
Contributor Author

@microsoft-github-policy-service agree

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit f4a060e:

✅ Validation status: passed

File Status Preview URL Details
articles/firewall/dns-settings.md ✅Succeeded

For more details, please refer to the build report.

@ttorble
Copy link
Contributor

ttorble commented Jul 31, 2025

@duongau

Can you review the proposed changes? Note that the author metadata should reflect your GitHub alias to facilitate automation. Can you create a PR in the private repo to update this? Thanks.

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 Jul 31, 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-dirichards
Copy link
Contributor

@duongau

Can you review this old PR and determine whether it needs to be closed or merged?

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

@duongau
Copy link
Contributor

duongau commented Aug 28, 2025

#sign-off

Copy link
Contributor

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

@v-dirichards
Copy link
Contributor

@duongau I'll merge this, thanks!

GitHub didn't recognize you as the author because your Microsoft alias of duau is in the metadata for both author and ms.author. You can update the author value in the private repo so that won't happen in the future.

@v-dirichards v-dirichards merged commit b0797b2 into MicrosoftDocs:main Aug 28, 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.

4 participants