Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds new user authentication method permissions for various authentication types including passkeys, passwords, phone, platform credentials, QR codes, software/hardware OATH, TAP, Windows Hello, email, external methods, and Microsoft Authenticator app. It appears to be part of a routine weekly permissions synchronization process.
- Adds comprehensive read/write permissions for multiple user authentication methods
- Includes both delegated and application permission schemes for most authentication types
- Maintains consistent structure and resource app ID across all new permission entries
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "id": "", | ||
| "scheme": "DelegatedWork", | ||
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "ea890292-c8c8-4433-b5ea-b09d0668e1a6" |
There was a problem hiding this comment.
The 'id' field is empty for all new permission entries. This could cause issues with permission identification and tracking. Each permission entry should have a unique identifier.
| "UserAuthMethod-Passkey.Read": [ | ||
| { | ||
| "id": "", | ||
| "scheme": "DelegatedWork", | ||
| "environment": "public", | ||
| "isHidden": true, | ||
| "isEnabled": true, | ||
| "resourceAppId": "ea890292-c8c8-4433-b5ea-b09d0668e1a6" | ||
| } | ||
| ], | ||
| "UserAuthMethod-Passkey.Read.All": [ | ||
| { | ||
| "id": "14195339-1fe4-48a7-a0d3-a39eb9fd8958", |
There was a problem hiding this comment.
There is a duplicate definition of 'UserAuthMethod-Passkey.Read.All' - one at line 15431 and another at line 16075. This creates conflicting permission entries that could cause runtime issues.
|
Closing this as a later PR added these permissions. |
Weekly Permissions sync 2025-08-06