Altamash Shaikh

@pboguslawski Have you set any cron to sync exiting users separately ? The code should not be called after login, I have checked the code again. This is the [code](https://github.com/matomo-org/plugin-LoginLdap/blob/5.x-dev/LdapInterop/UserSynchronizer.php#L160)...

@pboguslawski Are you able to debug this on your own ?

> roducing new events you should always consider that other plugins might want to use those events as well. > My approach here would have been to have an event...

@sgiehl I have updated the code as per your suggestion if all okay please don't merge as I need to get the text checked from product once to ensure if...

@sgiehl All good from product team to merge this :+1: Please review and merge this

@rcontiuk This looks like a good improvement :+1:

> > @haristku @AltamashShaikh I was able to do some functional testing today and things didn't seem to work quite as expected. No matter how many times I run `console...

@haristku We just had a chat on this internally, will it be possible for you to break this PR into 2-3 parts ? 1. getQueueIdForVisitor change 2. Redis cluster option...

> @AltamashShaikh @snake14 > > i have created two new PRs to separate the parts. PR for "redis cluster" part will follow later, because i have to fix the [**already...

@haristku Closing this one as we already splitted the PR into smaller PRs