-
Notifications
You must be signed in to change notification settings - Fork 44
Weekly Permissions sync 2026-01-11 #1401
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 pull request is a weekly permissions sync for 2026-01-11 that updates permission configurations across the platform.
Changes:
- Added new permissions for ManagedIdentity operations (Read, Update, Delete, HardDelete, Restore) and MS-ServicePrincipal.Create
- Added new Windows Update policy paths to support policies, approvals, and rings endpoints
- Fixed formatting and indentation inconsistencies throughout the permission definitions (trailing spaces, indentation alignment)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| permissions/new/provisioningInfo.json | Added new ManagedIdentity.* and MS-ServicePrincipal.Create permission entries; fixed formatting/indentation for multiple existing entries; corrected whitespace issues |
| permissions/new/permissions.json | Added new Windows Update paths for policies, approvals, and rings with appropriate HTTP method permissions (GET, POST, DELETE, PATCH) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "LockboxSettings.Read.All": [ | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| "LockboxSettings.ReadWrite.All": [ | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| "LockboxRequest.Read.All": [ | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ], | ||
| "LockboxRequest.ReadWrite.All": [ | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ] | ||
| { | ||
| "scheme": "DelegatedWork", | ||
| "environment": "PPE;public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-000000000000" | ||
| } | ||
| ] |
Copilot
AI
Jan 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Lockbox permission entries (LockboxSettings.Read.All, LockboxSettings.ReadWrite.All, LockboxRequest.Read.All, and LockboxRequest.ReadWrite.All) are missing the "id" field. For consistency with other permissions in this file that have specific environment values (like "PPE;public"), these entries should include an "id" field, even if it's empty. Other permissions with "PPE;public" environment (like AgentIdentityBlueprint permissions) include an "id" field.
Weekly Permissions sync 2026-01-11