Skip to content

Update description of Application Gateway mTLS auth server variable client_certificate_verification#128589

Open
ronaldbosma wants to merge 2 commits into
MicrosoftDocs:mainfrom
ronaldbosma:update-agw-client_certificate_verification-description
Open

Update description of Application Gateway mTLS auth server variable client_certificate_verification#128589
ronaldbosma wants to merge 2 commits into
MicrosoftDocs:mainfrom
ronaldbosma:update-agw-client_certificate_verification-description

Conversation

@ronaldbosma

Copy link
Copy Markdown
Contributor

Update description of Application Gateway mTLS auth server variable client_certificate_verification based on the behaviour I've observed (Standard_v2 AGW):

  • Passthrough mode:
    • NONE when no certificate is presented
    • PASSTHROUGH if a certificate is presented
  • Strict mode
    • SUCCESS when certificate is valid
    • Empty when certificate is invalid

…client_certificate_verification`

Update description of Application Gateway mTLS auth server variable `client_certificate_verification` based on the behaviour I've observed (Standard_v2 AGW):
- Passthrough mode: 
	- NONE when no certificate is presented
	- PASSTHROUGH if a certificate is presented
- Strict mode
	- SUCCESS when certificate is valid
	- Empty when certificate is invalid
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

1 similar comment
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 56d09fa:

✅ Validation status: passed

File Status Preview URL Details
articles/application-gateway/rewrite-http-headers-url.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot June 25, 2026 20:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the documentation for the Application Gateway mTLS auth server variable client_certificate_verification to reflect observed outputs in passthrough vs strict modes (notably for Standard_v2).

Changes:

  • Revised the client_certificate_verification server-variable description to enumerate values by mTLS mode.
  • Documented that strict mode may return an empty value when the client certificate is invalid (per observed behavior).

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

Comment thread articles/application-gateway/rewrite-http-headers-url.md Outdated
Comment thread articles/application-gateway/rewrite-http-headers-url.md Outdated
@v-dirichards

Copy link
Copy Markdown
Contributor

@mbender-ms

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 Jun 25, 2026
Clarified the description of client_certificate_verification for mTLS modes base on Copilot suggestions
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ab165a3:

✅ Validation status: passed

File Status Preview URL Details
articles/application-gateway/rewrite-http-headers-url.md ✅Succeeded

For more details, please refer to the build report.

@mbender-ms

Copy link
Copy Markdown
Contributor

#assign-reviewer: @JackStromberg

@mbender-ms

Copy link
Copy Markdown
Contributor

#unassign-reviewer: @JackStromberg

@prmerger-automator prmerger-automator Bot removed the request for review from JackStromberg June 29, 2026 14:01
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