Skip to content

Conversation

@marabooy
Copy link
Contributor

Weekly Permissions sync 2025-11-20

@marabooy marabooy requested a review from a team as a code owner November 20, 2025 00:33
Copilot AI review requested due to automatic review settings November 20, 2025 00:33
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 represents the weekly permissions sync for 2025-11-20, updating API endpoint permissions configurations in the Microsoft Graph permissions file.

Key Changes:

  • Refined permissions for /places/{id}/checkIns endpoints by separating them into dedicated DelegatedWork-only permission blocks
  • Added new claim providers API endpoints under Policy.Read.All and Policy.ReadWrite.ConditionalAccess permissions

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

Comment on lines 35600 to 35622
{
"schemeKeys": [
"DelegatedWork"
],
"methods": [
"POST"
],
"paths": {
"/identity/conditionalaccess/claimProviders": "",
"/identity/conditionalaccess/claimProviders/validate": ""
}
},
{
"schemeKeys": [
"DelegatedWork"
],
"methods": [
"DELETE",
"PATCH"
],
"paths": {
"/identity/conditionalaccess/claimProviders/{name}": ""
}
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The claim providers endpoints are added for POST (lines 35600-35610), DELETE, and PATCH (lines 35612-35622) methods in the Policy.ReadWrite.ConditionalAccess permission, but there's no corresponding GET method block for these endpoints. This is inconsistent with the pattern in Policy.Read.All (lines 34387-34397) which includes GET method support for /identity/conditionalaccess/claimProviders and /identity/conditionalaccess/claimProviders/getControls.

If Policy.ReadWrite.ConditionalAccess is meant to provide both read and write access to claim providers, a GET method block should be added similar to:

{
  "schemeKeys": [
    "DelegatedWork"
  ],
  "methods": [
    "GET"
  ],
  "paths": {
    "/identity/conditionalaccess/claimProviders": "",
    "/identity/conditionalaccess/claimProviders/getControls": "",
    "/identity/conditionalaccess/claimProviders/{name}": ""
  }
}

Copilot uses AI. Check for mistakes.
@jasonjoh jasonjoh closed this Dec 1, 2025
@jasonjoh jasonjoh deleted the permissions-update/2025-11-20 branch December 1, 2025 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants