Houdini / two_factor_authentication

Two factor authentication extension for Devise
MIT License
400 stars 270 forks source link

Make current migration safe using index concurrently #197

Closed Lackoftactics closed 4 years ago

Lackoftactics commented 4 years ago

The current approach is not great for bigger user bases. I added change to make it safer but to use algorithm: concurrently I had to disable transaction, which shouldn't be problem in our case.

Lackoftactics commented 4 years ago

@Houdini

Houdini commented 4 years ago

Thanks