Skip to content

Conversation

@michaelmsonne
Copy link
Contributor

@michaelmsonne michaelmsonne commented Nov 9, 2025

Make the Endpoint region requirement obvious as I see this can help to resolve when the endpoint doesn’t match the region of the Trusted Signing account/certificate profile. More clear for the eyes to see this requrement.

…bidden troubleshooting

Make the Endpoint region requirement obvious and provide a concrete example (403 Forbidden + SignerSign failure) with steps to resolve when the endpoint doesn’t match the region of the Trusted Signing account/certificate profile.
@prmerger-automator
Copy link
Contributor

@michaelmsonne : 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 59ee6aa:

✅ Validation status: passed

File Status Preview URL Details
articles/trusted-signing/how-to-signing-integrations.md ✅Succeeded

For more details, please refer to the build report.

@ktoliver
Copy link
Contributor

@TacoTechSharma

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 Nov 10, 2025
Removed the endpoint mismatch error example and its resolution steps from the document.

The error comes from signtool not Trusted signing and signtool team prefers to keep it generic.
@michaelmsonne
Copy link
Contributor Author

Updated RP after 1:1 chat with Meha.

@michaelmsonne michaelmsonne changed the title Highlight Trusted Signing endpoint region requirement and add 403 For… Highlight Trusted Signing endpoint region requirement Nov 16, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit aa6934f:

✅ Validation status: passed

File Status Preview URL Details
articles/trusted-signing/how-to-signing-integrations.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot November 17, 2025 21:07
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

This PR enhances documentation clarity by emphasizing the critical requirement that the Trusted Signing endpoint URI must match the region of both the account and certificate profile. The change converts a standard paragraph into an IMPORTANT callout box to make this requirement more visible to users.

Key Changes

  • Upgraded the endpoint region requirement from regular text to an IMPORTANT callout
  • Added explicit mention that the endpoint must match both the account and certificate profile region
  • Included troubleshooting context about 403 Forbidden errors caused by region/endpoint mismatches

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

@v-dirichards
Copy link
Contributor

@TacoTechSharma We'll merge this PR based on your approval. If this article requires further changes, please open a separate pull request.

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

This PR is ready for merge on November 18th.

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

@prmerger-automator
Copy link
Contributor

Are you trying to add/remove a custom label? The syntax for that is #label:"your label".

@v-dirichards
Copy link
Contributor

#label: "ready-to-merge"

@v-dirichards v-dirichards merged commit d223efa 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.

4 participants