Skip to content

Update app-service-ip-restrictions.md #126760

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Ramyaharinarthini
Copy link
Contributor

Added 162 line++ X-FD-HealthProbe is Custom header for identifying the health probe of the reverse proxy. Azure Front Door sends "1" to uniquely identify a health probe request. The header can also be used for non-Microsoft proxies to identify health probes. Accepts any string up to 64 characters in length.

Added 162 line++ X-FD-HealthProbe is Custom header for identifying the health probe of the reverse proxy. Azure Front Door sends "1" to uniquely identify a health probe request. The header can also be used for non-Microsoft proxies to identify health probes. Accepts any string up to 64 characters in length.
Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit d4629f1:

✅ Validation status: passed

File Status Preview URL Details
articles/app-service/app-service-ip-restrictions.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented May 9, 2025

@madsd

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 May 9, 2025
@madsd
Copy link
Contributor

madsd commented May 12, 2025

Hi @Court72, @Ramyaharinarthini. I don't think it makes sense just copying the description from one of the headers from this page (https://learn.microsoft.com/en-us/azure/app-service/overview-access-restrictions#http-header-filtering-for-site-access-restriction-rules) - better to change it to reference the specific section from the overview page.

Copy link
Contributor

@madsd madsd left a comment

Choose a reason for hiding this comment

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

I don't think it makes sense just copying the description from one of the headers from this page (https://learn.microsoft.com/en-us/azure/app-service/overview-access-restrictions#http-header-filtering-for-site-access-restriction-rules) - better to change it to reference the specific section from the overview page.

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.

3 participants