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

UserApi#listUserGroups() doesn't support pagination #1286

Closed
kamenkov opened this issue May 17, 2024 · 4 comments
Closed

UserApi#listUserGroups() doesn't support pagination #1286

kamenkov opened this issue May 17, 2024 · 4 comments
Assignees

Comments

@kamenkov
Copy link

Describe the feature request?

Currently, using the Java API, we cannot retrieve the full list of groups that a user is a member of. The standard REST API limit is 200 groups. The response headers contain a link to the next page. However, the Java API does not support pagination in this particular method. Therefore, we have to implement our own solution. It would be great if a PagedList could also be passed as a parameter in listUserGroups. Or, even better, in all the methods.

New or Affected Resource(s)

com.okta.sdk.resource.api.UserApi#listUserGroups(java.lang.String, java.util.Map<java.lang.String,java.lang.String>)

Provide a documentation link

https://developer.okta.com/okta-sdk-java/apidocs/com/okta/sdk/resource/api/UserApi.html#listUserGroups(java.lang.String,java.util.Map)

Additional Information?

No response

@arvindkrishnakumar-okta
Copy link
Contributor

PR #1506 should address this and please see the code snippets I've added to README.md to use Pagination in a controlled way.

@venkatesh-sw2
Copy link

Hi,

I was also waiting for this fix. This feature request was about providing a way for pagination of groups of a user ( which currently is not possible ). But the fix seems to be more on improving pagination of api to get users. Can you please look in to providing a way to retrieve the groups of a user.

@arvindkrishnakumar-okta
Copy link
Contributor

@venkatesh-sw2 Looks like you're referring to the below backend API:

image

This API does NOT support pagination in the backend and therefore the SDK will NOT support it too.

@biochimia
Copy link

I notice that 1a05092 just introduced support for paginating the listUserGroups endpoint in the API spec.

Is there a timeline you can share for cutting a release that includes the changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants