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

Reindexing of multiple users after database commit is not triggered #144

Open
SarahW91 opened this issue Sep 13, 2024 · 0 comments · May be fixed by inveniosoftware/flask-security-fork#62
Labels
bug Something isn't working

Comments

@SarahW91
Copy link

Package version (if known): 6.0.0

Describe the bug

When creating multiple users re-indexing is not triggered for all of them after committing to the data store. Users will be properly added to the database but not the search index.

Steps to Reproduce

  1. Create a loop in which multiple users are created using _datastore.create_user() with the datastore being a proxy from flask-security.
  2. Call _datastore.commit() after and outside of the loop.
  3. All users will be committed to the database but only the last created user will be indexed.

Expected behavior

All created users will be re-indexed.

Additional context

As far as I can see, after a user is created the database hook defined here should be triggered that re-indexes all updated users. However, at least for us this does not happen the way we expect it to.

@SarahW91 SarahW91 added the bug Something isn't working label Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant