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

Bug with MS.AAD.5.2v1 - "Only administrators SHALL be allowed to consent to applications." #1022

Closed
sgendron50 opened this issue Mar 26, 2024 · 8 comments · Fixed by #1043
Closed
Assignees
Labels
bug This issue or pull request addresses broken functionality public-reported This issue is reported by the public users of the tool.

Comments

@sgendron50
Copy link

🐛 Summary

What's wrong? Please be specific.

In the report, MS.AAD.5.2v1 is displaying "Fail", however, I've verified in Azure Portal --> Enterprise Apps --> Consent & Permissions, that "User consent for applications" is set to "Do not allow user consent". I've flipped it on and off, and ran the report again, but it still fails with:

1 authorization policies found that allow non-admin users to consent to third-party applications:
authorizationPolicy

Steps to reproduce the behavior:

Invoke-SCuBA -ProductNames aad

Expected behavior

I expected that 5.2 would pass.

Any helpful log output or screenshots

Paste the results here:

Add any screenshots of the problem here.

@buidav buidav added the public-reported This issue is reported by the public users of the tool. label Mar 26, 2024
@buidav
Copy link
Collaborator

buidav commented Mar 26, 2024

@tkol2022

@tkol2022
Copy link
Collaborator

@sgendron50 Thanks for reporting this. We are starting to see if we can re-produce it. @dagarwal-mitre

  • What type of tenant license do you have? E5, G5, G3, etc.
  • Is this a new problem?
  • Do you see any error messages in the terminal when running ScubaGear?

@sgendron50
Copy link
Author

@tkol2022 - We have an E5 license. I can't tell you if this is a new problem because we just used this for the first time. No, no error messages were displayed in the terminal. Let me know if there is anything else I can provide to assist.

@amandaw33
Copy link

We similarly hit this and similarly to @sgendron50 in Entra we do have user consent disabled, but over in admin center --> settings --> org settings --> services --> user consent to apps is checked, but I get an error trying to uncheck it (I thought I'd uncheck it to just stay consistent). "We couldn't save your changes. Close and reopen this setting to try again"

Googling hasn't led me to how this setting interacts or may be overridden by the Entra setting.

@DanielWahlgren
Copy link

We hit it aswell, and just started using it, so can't tell if it is new or not. We have E5 licenses, and have configured the settings seemingly correct in Entra. We were also unable to disable the setting in admin center like @amandaw33 pointed out.

Thank you for making this tool avaliable to the public.

@buidav buidav added the bug This issue or pull request addresses broken functionality label Mar 28, 2024
@tkol2022 tkol2022 linked a pull request Mar 29, 2024 that will close this issue
20 tasks
@tkol2022
Copy link
Collaborator

Thanks to everyone that reported and responded. Turns out that MS added some data to the API output which broke the policy check logic in the tool. The fix #1043 is currently in review so look for that soon.

@tkol2022
Copy link
Collaborator

We hit it aswell, and just started using it, so can't tell if it is new or not. We have E5 licenses, and have configured the settings seemingly correct in Entra. We were also unable to disable the setting in admin center like @amandaw33 pointed out.

Thank you for making this tool avaliable to the public.

Happy to know that people are using it! Appreciate the feedback.

@tkol2022
Copy link
Collaborator

tkol2022 commented Apr 2, 2024

FYI - The fix for this has been merged into the main branch. You can download the zip file from main if you want to test it - see screenshot below. It will also be included in our upcoming official release bundle.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue or pull request addresses broken functionality public-reported This issue is reported by the public users of the tool.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants