Cannot reactivate a deactivated account
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical SSO provider |
Fix Released
|
High
|
Szilveszter Farkas | ||
Bug Description
Once an account has been deactivated it cannot be used again unless an admin turns it back on.
Following the db split from lp, there is no longer a way for users to deactivate their own accounts so this currently only applies to accounts which were deactivated before the split. To reproduce this on production therefore requires a LOSA.
Steps to reproduce:
1. Deactivate your account from the admin UI.
2. Go to "Forgot password" and send yourself a password reminder.
3. Follow the link from the email you get sent.
4. Enter new password (and confirm it).
5. Failure.
What should happen:
...
4. A notification is displayed telling you that your account is currently deactivated
5. Enter new password (and confirm it).
6. Your account is reactivated.
7. The email you sent the reminder to is set as your preferred email.
8. A notification is displayed telling you about 6. and 7.
Notes:
* This should not work for suspended accounts! Suspended accounts must be reactivated by an administrator only.
Testcase: ISD_192
Related branches
- Ricardo Kirkner (community): Approve
-
Diff: 348 lines (+185/-21)7 files modified.bzrignore (+2/-0)
identityprovider/models/account.py (+8/-0)
identityprovider/templates/account/suspended.html (+17/-0)
identityprovider/tests/test_views_ui.py (+109/-8)
identityprovider/urls.py (+1/-0)
identityprovider/views/ui.py (+44/-9)
scripts/test (+4/-4)
| Changed in canonical-identity-provider: | |
| importance: | Undecided → Medium |
| Changed in canonical-identity-provider: | |
| status: | New → Confirmed |
| summary: |
- Cannot reactivate a deactivated address + Cannot reactivate a deactivated account |
| description: | updated |
| Changed in canonical-identity-provider: | |
| milestone: | none → 2.4.1 |
| Anthony Lenton (elachuni) wrote | #1 |
| Changed in canonical-identity-provider: | |
| importance: | Medium → High |
| milestone: | 2.4.1 → 2.5.0 |
| Changed in canonical-identity-provider: | |
| assignee: | nobody → Szilveszter Farkas (phanatic) |
| status: | Confirmed → In Progress |
| Changed in canonical-identity-provider: | |
| status: | In Progress → Fix Committed |
| Changed in canonical-isd-qa: | |
| assignee: | nobody → Dave Morley (davmor2) |
| Julien Funk (jaboing) wrote | #2 |
| Changed in canonical-isd-qa: | |
| status: | New → Confirmed |
| Dave Morley (davmor2) wrote | #3 |
| Changed in canonical-isd-qa: | |
| status: | Confirmed → Fix Committed |
| Dave Morley (davmor2) wrote | #4 |
| Changed in canonical-isd-qa: | |
| status: | Fix Committed → Fix Released |
| Changed in canonical-identity-provider: | |
| status: | Fix Committed → Fix Released |
| description: | updated |
We should also check, while doing this, that you're able to activate password-less accounts by requesting a password reminder.