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

Admin list identities - does not return result as expected with respect to the query parameters #2286

Closed
3 of 6 tasks
lbanders opened this issue Mar 4, 2022 · 3 comments
Closed
3 of 6 tasks
Assignees
Labels
area/docs This issue affects Ory's documentation.

Comments

@lbanders
Copy link

lbanders commented Mar 4, 2022

Preflight checklist

Describe the bug

As part of a user deletion process I want to list all the identities in kratos to find the proper id of my user.

According to the docs I can query the identies using /identities?page=0&per_page=3 to fetch the 3 identities in the database.

Given the above query and a database with 5 identities I expect I need to perform 2 queries:

  • /identities?page=0&per_page=3
  • /identities?page=1&per_page=3

However, the resultset from both these queries are alike, and I need to do a /identities?page=2&per_page=3 to get the last 2 identities in the list.

Reproducing the bug

  • Create 5 identities
  • Perform the queries as mentioned above on the admin interface

Relevant log output

No response

Relevant configuration

No response

Version

0.8.2

On which operating system are you observing this issue?

Linux

In which environment are you deploying?

Docker Compose

Additional Context

I am using a MySQL database for the users.

I have another service calling these endpoints.

@lbanders lbanders added the bug Something is not working. label Mar 4, 2022
@aeneasr
Copy link
Member

aeneasr commented Mar 4, 2022

page <=0 == page 1, basically we always start at 1

@aeneasr aeneasr self-assigned this Mar 4, 2022
@aeneasr aeneasr added area/docs This issue affects Ory's documentation. and removed bug Something is not working. labels Mar 4, 2022
aeneasr added a commit that referenced this issue Mar 4, 2022
@aeneasr aeneasr closed this as completed Mar 4, 2022
@lbanders
Copy link
Author

lbanders commented Mar 4, 2022

Ok - was just confused from the API documentation as it stated the page started from 0.

Thanks for clarifying :)

@aeneasr
Copy link
Member

aeneasr commented Mar 4, 2022

Yes, thank you for the report :) I fixed the API docs! :)

aeneasr added a commit that referenced this issue Mar 6, 2022
aeneasr added a commit that referenced this issue Mar 6, 2022
aeneasr added a commit that referenced this issue Mar 7, 2022
peturgeorgievv pushed a commit to senteca/kratos-fork that referenced this issue Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs This issue affects Ory's documentation.
Projects
None yet
Development

No branches or pull requests

2 participants