Weekly Permissions sync 2025-08-31#1251
Weekly Permissions sync 2025-08-31#1251jasonjoh merged 1 commit intopermissions-update/2025-08-30from
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR is a weekly permissions sync that updates API endpoint permissions in the system. The changes involve reorganizing and normalizing permission configurations for Teams and channel-related endpoints.
- Removes redundant or unnecessary permission entries from various pathSets
- Restructures permission schemes to better separate DelegatedWork and Application permissions
- Consolidates similar endpoint configurations to reduce duplication
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| "/teams/{id}/channels/{id}/doesuserhaveaccess": "least=DelegatedWork", | ||
| "/teams/{id}/channels/{id}/members": "least=DelegatedWork", |
There was a problem hiding this comment.
The permission values are inconsistent - some paths have 'least=DelegatedWork' while others in the same section have empty strings. Consider standardizing the permission format across similar endpoints for better maintainability.
| "/chats/{id}/members/{id}": "", | ||
| "/teams/{id}/channels/{id}/members/{id}": "" | ||
| } | ||
| "paths": {} |
There was a problem hiding this comment.
Empty paths object suggests this pathSet configuration may be redundant. Consider removing the entire pathSet block if no paths are defined, as it serves no functional purpose.
| ], | ||
| "paths": { | ||
| "/teams/{id}/channels/{id}/sharedwithteams/{id}": "least=DelegatedWork,Application" | ||
| "/teams/{id}/channels/{id}/sharedwithteams/{id}": "least=DelegatedWork" |
There was a problem hiding this comment.
The permission value has changed from 'least=DelegatedWork,Application' to 'least=DelegatedWork'. Verify this change doesn't break existing Application-level access patterns for shared team endpoints.
| "/teams/{id}/channels/{id}/sharedwithteams/{id}": "least=DelegatedWork" | |
| "/teams/{id}/channels/{id}/sharedwithteams/{id}": "least=DelegatedWork,Application" |
Weekly Permissions sync 2025-08-31