Users with duplicate emails being allowed

We have a growing number of users flowing in through social_auth (hundreds of thousands) and the issue of duplicate users has grown since originally reporting the issue.

We observed that the absence of the index cited in #5 🐛 Users with duplicate emails being allowed Active was slowing down the experience overall. Adding the index seems to speed things up substantially.

I suspect that there is still more to do to eliminate the root cause of the originally reported issue, but theoretically, speeding up queries for provider_user_id might help reduce the occurrence of the duplicates. I found that I could still reproduce the issue with the index in play, but it was much harder to replicate.

Any thoughts on this @wells?

bob.hinrichs → made their first commit to this issue’s fork.

There is a new merge request here that introduces using Drupal::lock() to prevent running authenticateUser() in parallel for the same user. We have not yet made this change live, will do so 1/5/24.

I've also sometimes getting duplicate users when user registering with google. Using patch from #8 and will get back tommorrow to see the result