-
Notifications
You must be signed in to change notification settings - Fork 438
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
Drupal: Harden email change procedure #2566
Drupal: Harden email change procedure #2566
Conversation
When changing an email, use receives notification at new and old email address. Add checks for previous email address. Also add check for boincuser_delete account to terms of use form. Matches UX from upstream: BOINC#2451 Resolves: https://dev.gridrepublic.org/browse/DBOINCP-436 Also see: * BOINC#2500 * https://boinc.berkeley.edu/trac/wiki/EmailChangeNotification
User may recovert previous email by using a form. This will revert the user's email but s/he must set a new password first.
User may not delete account until seven days after an email change. Additionally, delete account form has a link to the email/password change form.
create_account RPC changed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small bug, otherwise looks ok to me.
@@ -1555,6 +1630,13 @@ function boincuser_create_account() { | |||
} | |||
|
|||
// Process input | |||
// Check this email against previous email addresses. | |||
$tmpuser = BoincUser::lookup_pr3ev_email_addr($params['email_addr']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo in function name, probably won't work. 😄
I managed to forgot a |
This code change will make the Drupal web code match the upstream Web code (PR #2500) for user's email address change. The workflow is described in the boinc wiki.
https://dev.gridrepublic.org/browse/DBOINCP-436