Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FR] - Feedback for the MSEntraID Responder #1302

Open
stacsirt opened this issue Dec 18, 2024 · 1 comment
Open

[FR] - Feedback for the MSEntraID Responder #1302

stacsirt opened this issue Dec 18, 2024 · 1 comment

Comments

@stacsirt
Copy link

Hi There:

I noticed in step 7 of the setup instructions that you recommend adding the Directory.ReadWrite.All and User.ReadWrite.All API permissions. However, I tested the analyzer in my tenant and found that Directory.Read.All (Microsoft Graph API, application permission) works perfectly fine. There’s no need for the write permissions in this case.

I hope this helps clarify the requirements and potentially simplify the setup for others.

Regards,
Lyle

@nusantara-self
Copy link
Contributor

Hello Lyle,

Thanks for notifying this to us! I think there might have been a confusion with the responder documentation. I have just checked and the documentation of the analyzer mentions the use of Directory.Read.All, AuditLog.Read.All, and Policy.Read.ConditionalAccess permissions for the analyzer.

However, the responder, recommends Directory.ReadWrite.All and User.ReadWrite.All, as it signs out a user / revoke all tokens.

If you find that any of the recommended permissions are more permissive than necessary for certain use cases, please let us know and we would be happy to update the documentation accordingly.

Have a great day ahead,
Fabien

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

No branches or pull requests

2 participants