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

Pagination is wrong when listing consent session on Ory/Hydra #3789

Open
5 tasks done
supercairos opened this issue Jun 26, 2024 · 0 comments
Open
5 tasks done

Pagination is wrong when listing consent session on Ory/Hydra #3789

supercairos opened this issue Jun 26, 2024 · 0 comments
Labels
bug Something is not working.

Comments

@supercairos
Copy link

Preflight checklist

Ory Network Project

No response

Describe the bug

When listing all consent session, Hydra filters out the expired consent sessions.

return p.filterExpiredConsentRequests(ctx, rs)

But when counting the availlable consent sessions, Hydra doesn't filter out the expired consent sessions.

return n, sqlcon.HandleError(err)

Leading to an inconsistency between the pagination and the returned consent session.

Reproducing the bug

Have some expired session in your DB.

List all sessions. notice the pagination header is wrong.

Relevant log output

No response

Relevant configuration

Ory/Hydra 2.2.0

Version

2.2.0

On which operating system are you observing this issue?

Linux

In which environment are you deploying?

Kubernetes with Helm

Additional Context

No response

@supercairos supercairos added the bug Something is not working. label Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

No branches or pull requests

1 participant