microsoft / microsoft/AzureTRE

Investigate minimal AAD API permissions - such as User.ReadBasic.All

Open
#384 1 comment 0 reactions 1 assignee View on GitHub

@marrobi is already working on this.

Since Nov 25, 2025.

api enhancement story
Dominant language
Python
Stars
235
Forks
192
Avg merge
1d 23h
Merged PRs (30d)
13

Description

Description

When app registration are created we want to do this with the lowest possible permissions to reduce perceived risk.

We are using Directory.Read.All - this is required to work out which workspaces/app registrations a user is a member off. Can we do this another way? - If we add groups to the app reg roles - could use GroupMember.Read.All - this call is deemed low risk.

Or as a start I believe: Application.Read.All and AppRoleAssignment.ReadWrite.All will work instead of Directory.Read.All and User.ReadBasic.All.

The graph call is only needed for listing workspaces. In certain circumstances where there is no requirement to list workspaces the users could be given URL to the workspace without the need to list all workspaces - or use the my app portal.

See: (https://docs.microsoft.com/en-us/graph/permissions-reference)[https://docs.microsoft.com/en-us/graph/permissions-reference]

Acceptance criteria

  • Minimal permissions figured out
  • Permissions changes implemented
  • Changes updated in /docs/auth.md and other relevant pieces of documentation

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.