Microsoft Connector: Update group scope to groupMember.read.all

Open ap0phi5 opened this issue 9 months ago • 5 comments

Overview

This is a small change to reduce the default permission scope for a Microsoft connector from the overly-permissive directory.read.all down to groupMember.read.all.

What this PR does / why we need it

Closes #3989

Mar 13 '25 14:03 ap0phi5

Anyone able to allow this to run the CI tests?

Mar 15 '25 09:03 ap0phi5

@nabokihms If it is within your purview, could you let the tests run on this please? The details look a bit old but I'm assuming this tenant is still available? https://github.com/dexidp/dex/blob/master/connector/microsoft/microsoft_test.go#L20-L23

Mar 24 '25 14:03 ap0phi5

Bump. Is there anything more I can do to help this PR be considered?

Jun 24 '25 11:06 ap0phi5