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

100 users limitation #64

Open
sergeycherepanov opened this issue Mar 7, 2017 · 2 comments
Open

100 users limitation #64

sergeycherepanov opened this issue Mar 7, 2017 · 2 comments

Comments

@sergeycherepanov
Copy link

sergeycherepanov commented Mar 7, 2017

Hi!

Currently plugin has limitation in max 100 users, and if your have more than 100 user, plugin works incorrect.

We have 100+ users in our ES cluster and some users can't login. It was a suddenly for us.

I think need to add information about this limit to readme.

Issue in this line: https://github.com/elasticfence/elasticsearch-http-user-auth/blob/2.4.1/src/main/java/org/elasticsearch/plugin/elasticfence/data/UserDataBridge.java#L322

@lmangani
Copy link
Collaborator

Thanks @sergeycherepanov feel free to send a PR to change this or create a parameter for it

@sergeycherepanov
Copy link
Author

Hi @lmangani
I would gladly to do this, but my java and elastic experience is to low for this. I think here is needed some changes in logic. Storing the all user data in memory maybe not good solution. Would be better to implement lazy load with caching.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants