Skip to content

Weekly Permissions sync 2025-08-21 - #1232

Closed
David (marabooy) wants to merge 1 commit into
devfrom
permissions-update/2025-08-21
Closed

David (marabooy) wants to merge 1 commit into
devfrom
permissions-update/2025-08-21

Conversation

@marabooy

Copy link
Copy Markdown
Contributor

Weekly Permissions sync 2025-08-21

Copilot AI review requested due to automatic review settings August 21, 2025 00:30
@marabooy
David (marabooy) requested a review from a team as a code owner August 21, 2025 00:30

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

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"

Copilot AI Aug 21, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
"resourceAppId": "00000003-0000-0000-c000-00000000000"
"resourceAppId": "00000003-0000-0000-c000-000000000000"

Copilot uses AI. Check for mistakes.
"environment": "public",
"isHidden": true,
"isEnabled": false,
"resourceAppId": "00000003-0000-0000-c000-00000000000"

Copilot AI Aug 21, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
"resourceAppId": "00000003-0000-0000-c000-00000000000"
"resourceAppId": "00000003-0000-0000-c000-000000000000"

Copilot uses AI. Check for mistakes.
"environment": "public",
"isHidden": true,
"isEnabled": false,
"resourceAppId": "00000003-0000-0000-c000-00000000000"

Copilot AI Aug 21, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
"resourceAppId": "00000003-0000-0000-c000-00000000000"
"resourceAppId": "00000003-0000-0000-c000-000000000000"

Copilot uses AI. Check for mistakes.
"environment": "public",
"isHidden": true,
"isEnabled": false,
"resourceAppId": "00000003-0000-0000-c000-00000000000"

Copilot AI Aug 21, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
"resourceAppId": "00000003-0000-0000-c000-00000000000"
"resourceAppId": "00000003-0000-0000-c000-000000000000"

Copilot uses AI. Check for mistakes.
"environment": "public",
"isHidden": true,
"isEnabled": false,
"resourceAppId": "00000003-0000-0000-c000-00000000000"

Copilot AI Aug 21, 2025

Copy link

Choose a reason for hiding this comment

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

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.

Suggested change
"resourceAppId": "00000003-0000-0000-c000-00000000000"
"resourceAppId": "00000003-0000-0000-c000-000000000000"

Copilot uses AI. Check for mistakes.
@jasonjoh
Jason Johnston (jasonjoh) force-pushed the permissions-update/2025-08-21 branch from e143692 to 368899f Compare August 28, 2025 20:49
@jasonjoh

Copy link
Copy Markdown
Member

Closing as remaining changes seem to undo changes made 2 days ago in #1239

@jasonjoh
Jason Johnston (jasonjoh) deleted the permissions-update/2025-08-21 branch August 28, 2025 20:50
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