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

Broader usage of Paginated Lists #271

Open
6 tasks
cipherboy opened this issue Apr 6, 2024 · 2 comments
Open
6 tasks

Broader usage of Paginated Lists #271

cipherboy opened this issue Apr 6, 2024 · 2 comments
Labels
feature good first issue Good for newcomers help wanted Extra attention is needed

Comments

@cipherboy
Copy link
Contributor

cipherboy commented Apr 6, 2024

Is your feature request related to a problem? Please describe.

In #140 and #170, initial support of paginated lists were added, which included some API endpoints. However, some more endpoints could be updated to use these new APIs.

Describe the solution you'd like

  • PKI Tidy operation should use paginated lists everywhere.
  • Audit other tidy operations for usage of paginated lists
  • Lease expiry should use paginated lists
  • Token expiry should use paginated lists
  • Secrets and auth engine listing should probably support pagination as well. This does not use a storage.List call.
  • Additional calls to storage.List should probably be audited as well.

Describe alternatives you've considered

Some places will continue to rely on storage.List as they have a low volume of responses, e.g., issuer counts should be in the 10s at most.

Explain any additional use-cases

n/a

Additional context

This will help with memory utilization in more high-volume areas of the code. Would be a good GA milestone candidate.

@naphelps naphelps added good first issue Good for newcomers help wanted Extra attention is needed labels Apr 8, 2024
@lplazas
Copy link

lplazas commented Apr 30, 2024

I'd be happy to get this assigned to me.

@cipherboy
Copy link
Contributor Author

@lplazas I think a great place to start would be the PKI engine. While large, I think the changeset should mostly be limited to https://github.com/openbao/openbao/blob/main/builtin/logical/pki/path_tidy.go :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants