Skip to content

Weekly Permissions sync 2026-04-08#1496

Closed
marabooy wants to merge 2 commits intomasterfrom
permissions-update/2026-04-08
Closed

Weekly Permissions sync 2026-04-08#1496
marabooy wants to merge 2 commits intomasterfrom
permissions-update/2026-04-08

Conversation

@marabooy
Copy link
Copy Markdown
Contributor

@marabooy marabooy commented Apr 8, 2026

Weekly Permissions sync 2026-04-08

@marabooy marabooy requested a review from a team as a code owner April 8, 2026 00:29
Copilot AI review requested due to automatic review settings April 8, 2026 00:29
Copy link
Copy Markdown
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

Weekly permissions sync updating the permissions dataset under permissions/new/ by adding new permission variants and adjusting endpoint-to-permission mappings.

Changes:

  • Added new permission entries to provisioningInfo.json (AgentIdentityBlueprintPrincipal.UpdateSponsors.All, OneDrive.Provision.All, ServicePrincipal.*).
  • Added /security/dataSecurityAndGovernance/processContent to the Content.Process.All path set in permissions.json.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
permissions/new/provisioningInfo.json Adds new permission variant records for additional scopes/roles.
permissions/new/permissions.json Adds a missing /security/.../processContent path mapping under Content.Process.All.

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

Comment on lines +9827 to +9834
"OneDrive.Provision.All": [
{
"id": "00000000-0000-0000-0000-000000000000",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": true,
"resourceAppId": "00000003-0000-0ff1-ce00-000000000000"
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

OneDrive.Provision.All uses an all-zero GUID for the id field. Elsewhere in this file, unknown/placeholder permission IDs are represented as an empty string (""), so using 00000000-0000-0000-0000-000000000000 is inconsistent and may be treated as a real ID by downstream tooling. Please replace this with an empty string if the real permission ID is unknown, or populate the actual GUID if available.

Copilot uses AI. Check for mistakes.
"resourceAppId": "00000002-0000-0000-c000-000000000000"
}
],
"AgentRegistration.Read.All": [
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

There is an extra leading space before the "AgentRegistration.Read.All" key compared to surrounding entries, which breaks the otherwise consistent indentation/formatting of this JSON file. Please align this key's indentation with the other top-level permission keys in the same object.

Suggested change
"AgentRegistration.Read.All": [
"AgentRegistration.Read.All": [

Copilot uses AI. Check for mistakes.
@jasonjoh jasonjoh closed this Apr 10, 2026
@jasonjoh jasonjoh deleted the permissions-update/2026-04-08 branch April 10, 2026 21:01
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