Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify new users by email #1535

Closed
rossjones opened this issue Feb 25, 2014 · 5 comments
Closed

Verify new users by email #1535

rossjones opened this issue Feb 25, 2014 · 5 comments

Comments

@rossjones
Copy link
Contributor

Support for email verification of new user accounts would be very useful for systems that want to ensure the email account provided actually exists/is-valid.

@davidread
Copy link
Contributor

Aside from the anti-spam element, it's also pretty necessary to allow people to recover their passwords.

@adamamyl
Copy link
Contributor

+1, although (from my perspective) it would be useful for an admin to be able to disable the feature.

@davidread
Copy link
Contributor

sean: this is a persona ckan extensino which already does this. nigel/adria: keen to have this in core. user would need a new column for the key.

@rufuspollock
Copy link
Member

@rossjones @davidread might be worth detailing the implementation and UX steps here. I imagine something like:

DB

  • Add field confirmation_token to user table.
    • confirmation_token just a uuid
    • If confirmation_token is not null then account is not usable [alternative: have a separate status field which is set to "awaiting confirmation" or similar]

Logic / User Flow

  • Upon sign-up user has confirmation_token set. They are sent email with confirmation token link /user/confirm?token=....
    • confirmation_token also set on password sign-up (??) - not sure this is necessary (could just send that an email saying their password was reset)
  • New controller action /user/confirm
    • If correct, set confirmation_token to null and redirect user to their dashboard
    • If failure notify and leave token as is

@rossjones
Copy link
Contributor Author

There's a new issue for this ( #2838 ) where a PR will be incoming. Closing this in favour of that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants