yii2-usuario icon indicating copy to clipboard operation
yii2-usuario copied to clipboard

Published 20 hours ago • verified publisher icon 2amigos

Update PasswordRecoveryService to check if the user exist before send…

Open Sara23 opened this issue 4 years ago • 3 comments

… email.

Q A
Is bugfix? yes
New feature? yes
Breaks BC? no
Fixed issues Check if the user exist before send email.

Sara23 avatar Jun 02 '21 18:06 Sara23

Why this change? Can't you trap RuntimeException and handle the error in the application, can you?

maxxer avatar Jun 03 '21 09:06 maxxer

I do not like the mechanics of being able to check if an user exists or not. I think the default "blind" way is good and should not be changed unless any serious arguments are presented.

TonisOrmisson avatar Jun 03 '21 10:06 TonisOrmisson

Agree with @TonisOrmisson. If you want this change to be merged, it should be controlled by a configuration parameter, but the default behavior must be the current one.

maxxer avatar Jul 18 '22 11:07 maxxer