Skip to content

Weekly Permissions sync 2026-06-08#1553

Open
marabooy wants to merge 1 commit into
masterfrom
permissions-update/2026-06-08
Open

Weekly Permissions sync 2026-06-08#1553
marabooy wants to merge 1 commit into
masterfrom
permissions-update/2026-06-08

Conversation

@marabooy

@marabooy marabooy commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Weekly Permissions sync 2026-06-08

@marabooy marabooy requested a review from a team as a code owner June 8, 2026 00:29
Copilot AI review requested due to automatic review settings June 8, 2026 00:29

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

Weekly permissions sync update adjusting permission-to-path mappings and provisioning metadata in the permissions/new dataset.

Changes:

  • Updates several permission paths mappings (adds/removes/adjusts least= and AlsoRequires= qualifiers, adds new endpoints under existing scopes).
  • Removes the Policy.Read.ApplicationConfiguration permission definition from permissions.json.
  • Adds a new (hidden/disabled) Policy.Read.ApplicationConfiguration entry in provisioningInfo.json.

Reviewed changes

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

File Description
permissions/new/provisioningInfo.json Adds provisioning entries for Policy.Read.ApplicationConfiguration (hidden/disabled).
permissions/new/permissions.json Updates multiple scope-to-path mappings and removes the Policy.Read.ApplicationConfiguration permission definition.
Comments suppressed due to low confidence (1)

permissions/new/permissions.json:38531

  • Policy.Read.ApplicationConfiguration was removed from permissions.json, but this PR still introduces/keeps it in provisioningInfo.json. If the scope is still supported, its permission metadata (schemes, descriptions, pathSets, ownerInfo) needs to exist here; if it’s being retired, the provisioning entry should be removed instead. As-is, the two files disagree about whether this permission exists.

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

Comment on lines +11511 to +11518
"Policy.Read.ApplicationConfiguration": [
{
"id": "",
"scheme": "Application",
"environment": "public",
"isHidden": true,
"isEnabled": false,
"resourceAppId": "00000002-0000-0000-c000-000000000000"
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.

2 participants