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

Investigate glauth sql backend usage #59

Open
nsklikas opened this issue Aug 27, 2024 · 1 comment
Open

Investigate glauth sql backend usage #59

nsklikas opened this issue Aug 27, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@nsklikas
Copy link
Contributor

Following up the discussion on #55 (comment).

The issue is that right now the postgres backend (I assume all backends) fetch all the users from the database and filter them in memory. There are 2 things we can approach on this:

  1. Add more logic when constructing the db query, IF we are trying to search for a user based on cn we should let the database to do the filtering and use the power of its existing indexes
  2. Add a jsonb index that allows us to perform more complex search operations based on random attributes

(1) is easier to implement and should result in a speed up in most cases, (2) will be harder to implement as we will have to check the database version and may not be used that much.

When we have more real world usage of glauth with a backend we will need to evaluate whether we need to refactor the database logic.

@nsklikas nsklikas added the enhancement New feature or request label Aug 27, 2024
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/IAM-1010.

This message was autogenerated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant