100 users limitation - Githubissues

elasticfence / elasticsearch-http-user-auth

HTTP Basic Authentication and IP ACL plugin for Elasticsearch :key: (UNMAINTAINED)

http://qxip.net

148 stars 32 forks source link

100 users limitation #64

Open sergeycherepanov opened 7 years ago

sergeycherepanov commented 7 years ago

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 commented 7 years ago

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

sergeycherepanov commented 7 years ago

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.