get_group_memberships_from_ms_graph returns only first 100 users

[freyp577]

It looks like the method get_group_memberships_from_ms_graph in backend.py returns only the first 100 users
Unfortunatley we have usrs that have over 200 groups assigned

we are using django-auth-adfs with Azure AD
and the Microsft Graph API results have a size limit (of 100, in our case)

requiring the groups exceeding the limit
need to be fetched by repeating the reqest with '@odata.nextLink'
that is returned in this case with the batched result

could this be fixed / extended?

[JonasKs]

PR welcome 😊

[tim-schilling]

Alternatively, you can subclass the ADFS backend and override get_group_memberships to do what you need it to.

[freyp577]

I am preparing a PR ...
currently testing it

[freyp577]

PR is hiere:

#274

[guillaumeldc]

Hi,

Where did we leave off with this request? I do have the same issue, believe it or not, I found use cases where users have 500+ groups in AD and won't want to customize the claim to return the groups attached to the application.
Please let me know if a PR is still needed but it seems like @freyp577 proposed a sound solution? @JonasKs what do you think?

[tim-schilling]

There's an open PR but it has some things that need to be addressed. You can base your work off of that.