FOSUserBundle icon indicating copy to clipboard operation
FOSUserBundle copied to clipboard

When changing email disable account and verify the new email

Open runcom opened this issue 12 years ago • 7 comments

I think this should be done out-of-the-box in order to always have valid email addresses (if not why verifing only the first email?!) Would be really nice if implemented

runcom avatar Jan 14 '13 21:01 runcom

Please be a bit patient. You opened it only 17 hours ago. I have a life outside the Symfony community too, and a job.

And btw, I agree that this feature could be useful. Do you mind contributing to implement it ?

stof avatar Jan 15 '13 14:01 stof

@stof stof found a solution (without disabling the account)

in /profile/edit when a user change his email display in the profile/edit page that a link for verification has been sent to his new email waiting for activation and keep the "old" email as main email till he verifies the new email This procedure will be done if registration:confirmation is set to true in cofig, otherwise do as usual

I think that the above requires a db column for email_verified too

How you see this?

runcom avatar Jan 16 '13 18:01 runcom

@amurdaca Where is the PR?

b-durand avatar May 26 '13 10:05 b-durand

Can you supply a PR @runcom ?

core23 avatar Sep 28 '16 17:09 core23

I was thinking about this issue and a few thoughts cropped up. Locking users out of their account because they changed their email would be unexpected behavior for most projects and FOSUserBundle doesn't give any feedback why you can't access your account, nor does it currently offer the user to initiate a repeat email in case something went wrong.

We could make it optional to disable the account if the email is not verified, that would make it acceptable. I think that implementing this feature correctly would mean that an old email address should be saved as long as a new email address is in review. Maybe this feature is more suited for a user bundle with more advanced security policies?

codedmonkey avatar Oct 03 '16 19:10 codedmonkey

I hope this feature implemented

biji avatar Mar 01 '18 12:03 biji

#2612 (including the "update email confirmation"-feature) was unfortunately reverted. For anyone still interested in the "update email confirmation" feature: as FOSUserBundle does not move, I we implemented this feature as separate bundle. See https://github.com/azine/AzineEmailUpdateConfirmationBundle

azine avatar Mar 19 '19 10:03 azine