authboss icon indicating copy to clipboard operation
authboss copied to clipboard

Strange behaviour if user recover the account before the confirmation

Open kenng opened this issue 9 years ago • 5 comments

After registration and before confirming the account, if the user recover the account via the link, the user will be auto log in, this is alright. If the user log out now, and try to login again he will be prompted to confirm his account again. So, probably we can consider the user account is confirmed if he is able to recover it?

kenng avatar Jun 03 '16 02:06 kenng

I think this makes sense. Because confirming and recovering both use the e-mail. Were you looking for us to fix this? Or were you going to submit a PR?

aarondl avatar Jun 08 '16 04:06 aarondl

It could be a little bit strange though looking at the code. It sort of violates the isolation we've got between the storage and the modules. Currently you can use one without the other, but this almost creates a dependency.

aarondl avatar Jun 08 '16 04:06 aarondl

Haven't got the time to dive deep into the code, but what i remembered seeing there is a module function which will load each modules one by one. During the module registration, is there any possibility to register a callback function based on event? Then recover module can publish the event for confirm module to handle?

kenng avatar Jun 28 '16 10:06 kenng

Then it would depend on which module was loaded first unfortunately. :(

aarondl avatar Sep 07 '16 04:09 aarondl

Looking at this again, it's possible now in the v2 code to implement this properly using events. Ordering should not be a concern.

aarondl avatar May 14 '18 22:05 aarondl