feat: support Entra guest user groups #310
Conversation
|
Refactored the Microsoft API calls into an "advanced query" which allows filtering with Filtering groups by the |
charn
force-pushed
the
entra-guest
branch
2 times, most recently
from
84871fa
to
714fad1
Compare
|
Since security enabled groups are now filtered by Microsoft API we can also fix |
Entra guest users have limited visibility into groups via the API. Basically they can only see the group id's they belong to. Both displayName and securityEnabled values are null. However filtering in the request using ``$filter=securityEnabled eq true` still works for guest users. This feature allows returning group UUIDs for guest users. Filtering groups by the `securityEnabled` flag requires using an "advanced query", which requires adding `$count=true` query parameter and a `ConsistencyLevel: eventual` HTTP header. Since security enabled groups are now filtered by Microsoft API we can also fix `TestADGroupsFromGraphResponse` accordingly. Refs: HP-2434
charn
force-pushed
the
entra-guest
branch
2 times, most recently
from
a108b3d
to
a0ab5f5
Compare
Also: - update GitHub Action versions - Use codecov action - Cache pip with setup-python action
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #310 +/- ##
===========================================
+ Coverage 80.29% 80.35% +0.06%
===========================================
Files 91 91
Lines 3775 3787 +12
===========================================
+ Hits 3031 3043 +12
Misses 744 744 ☔ View full report in Codecov by Sentry. |
Entra guest users have limited visibility into groups via the API. Basically they can only see the group id's they belong to. Both displayName and securityEnabled values are null. However filtering in the request using
$filter=securityEnabled eq truestill works for guest users. This feature allows returning group UUIDs for guest users.Refs: HP-2434