Skip to content

Conversation

NoahJenkins
Copy link

This pull request adds an important clarification to the "Conditional Access for workload identities" documentation.

It addresses a common scenario where administrators might add a service principal to a group and assume that any Conditional Access policies targeting that group will apply to the service principal. This is not the case.

This change adds a [!NOTE] to explicitly state that Conditional Access policies are not enforced on service principals through group membership. To be effective, the policy must be assigned directly to the service principal as a workload identity. This helps prevent misconfiguration and ensures administrators understand the correct procedure for securing workload identities.

…oup are not enforced for service principals within that group. To apply a policy to a service principal, it must be assigned directly as a workload identity.
@Copilot Copilot AI review requested due to automatic review settings August 26, 2025 19:28
Copy link

@Copilot 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 adds a clarification note to the workload identity documentation to address a common misconception about Conditional Access policy enforcement for service principals.

  • Adds a [!NOTE] section clarifying that service principals in groups are not subject to group-targeted Conditional Access policies
  • Explains that policies must be assigned directly to service principals as workload identities to be effective

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

Copy link
Contributor

@NoahJenkins : 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 b0866f7:

✅ Validation status: passed

File Status Preview URL Details
docs/identity/conditional-access/workload-identity.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards
Copy link
Contributor

@MicrosoftGuyJFlo

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

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