Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR performs a weekly permissions synchronization, updating permission definitions and deployment configurations across the Microsoft Graph permissions system. The changes include removing deprecated agent-related permissions, updating authentication method permissions with better granularity, and adding new permissions for various services.
Key changes:
- Removed deprecated
AgentApplication.CreateandAgentIdentity.Createpermissions - Restructured user authentication method permissions to provide more granular access control
- Added new permissions for backup/restore, security, and zone management services
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| permissions/new/permissions.json | Updated permission definitions, removed agent permissions, restructured auth method permissions, and added new API endpoints |
| permissions/new/ProvisioningInfo.json | Updated deployment configurations for new and existing permissions, including visibility and resource app assignments |
Comments suppressed due to low confidence (4)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
| "isHidden": false, | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId appears to be missing the final digit. It should be "00000003-0000-0000-c000-000000000000" (with 12 zeros at the end) to match the standard Microsoft Graph App ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": false, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId appears to be missing the final digit. It should be "00000003-0000-0000-c000-000000000000" (with 12 zeros at the end) to match the standard Microsoft Graph App ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": false, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId appears to be missing the final digit. It should be "00000003-0000-0000-c000-000000000000" (with 12 zeros at the end) to match the standard Microsoft Graph App ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": false, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId appears to be missing the final digit. It should be "00000003-0000-0000-c000-000000000000" (with 12 zeros at the end) to match the standard Microsoft Graph App ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": false, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId appears to be missing the final digit. It should be "00000003-0000-0000-c000-000000000000" (with 12 zeros at the end) to match the standard Microsoft Graph App ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
jasonjoh
force-pushed
the
permissions-update/2025-08-16
branch
from
5d828f6 to
90c1cb1
Compare
Weekly Permissions sync 2025-08-16