Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR represents a weekly permissions sync for 2025-08-21, updating permissions configurations across Microsoft Graph API endpoints. The changes include removal of deprecated permissions, addition of new endpoint paths, updates to permission scopes, and extensive refactoring of authentication method permissions.
- Removal of deprecated AgentApplication and AgentIdentity permissions
- Addition of new API endpoints for backup/restore, virtual events, network access reports, and other services
- Major restructuring of user authentication method permissions with granular permission names
- Updates to permission deployment configurations and resource app IDs
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| permissions/new/permissions.json | Updates API endpoint permissions, removes deprecated permissions, adds new paths, and restructures authentication method permissions |
| permissions/new/ProvisioningInfo.json | Updates permission deployment configurations, adds new authentication method permissions, and corrects resource app IDs |
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.
| "isHidden": false, | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "00000003-0000-0000-c000-00000000000" |
There was a problem hiding this comment.
The resourceAppId value is missing the final '0' digit. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource 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 value is missing the final '0' digit. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource 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 value is missing the final '0' digit. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource 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 value is missing the final '0' digit. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource 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 value is missing the final '0' digit. It should be '00000003-0000-0000-c000-000000000000' to match the standard Microsoft Graph resource app ID format.
| "resourceAppId": "00000003-0000-0000-c000-00000000000" | |
| "resourceAppId": "00000003-0000-0000-c000-000000000000" |
jasonjoh
force-pushed
the
permissions-update/2025-08-21
branch
from
e143692 to
368899f
Compare
|
Closing as remaining changes seem to undo changes made 2 days ago in #1239 |
Weekly Permissions sync 2025-08-21