SSO secondary password shouldn't say didn't match when there is an issue with the first
Bug #579920 reported by
Dave Morley
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical SSO provider |
Fix Released
|
Low
|
David Owen | ||
Bug Description
Steps To Reproduce:
1. Login into http://
2. Modify your password to Test1 in the 2 password boxes
3. Note that the warning on the top password is correct there weren't 8 letters
4. Note also there is a didn't match password on the second box
I'm assuming this is because the first password got emptied when there was the less than 8 character issue was tripped and so the second reported that it didn't match the first as it was now empty.
I believe the easiest way round this is to simply tell the second not to display a fault if one is already displayed.
Related branches
lp:~canonical-isd-hackers/canonical-identity-provider/bug_579920
- Danny Tamez: Pending requested
-
Diff: 41 lines (+21/-0)2 files modifiedidentityprovider/forms.py (+1/-0)
identityprovider/tests/test_forms.py (+20/-0)
| Changed in canonical-isd-qa: | |
| assignee: | nobody → Dave Morley (davmor2) |
Revision history for this message
| Stuart Metcalfe (stuartmetcalfe) wrote | #1 |
| Changed in canonical-identity-provider: | |
| status: | New → Confirmed |
| importance: | Undecided → Low |
| milestone: | none → 2.6.0 |
| Changed in canonical-identity-provider: | |
| assignee: | nobody → David Owen (dsowen) |
| summary: |
- SSO secondary password should say didn't match when there is an issue + SSO secondary password shouldn't say didn't match when there is an issue with the first |
| Changed in canonical-identity-provider: | |
| status: | Confirmed → In Progress |
| Changed in canonical-identity-provider: | |
| status: | In Progress → Fix Committed |
| Changed in canonical-identity-provider: | |
| status: | Fix Committed → In Progress |
| Changed in canonical-identity-provider: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Dave Morley (davmor2) wrote | #2 |
| Changed in canonical-isd-qa: | |
| status: | New → Confirmed |
Revision history for this message
| Dave Morley (davmor2) wrote | #3 |
| Changed in canonical-isd-qa: | |
| status: | Confirmed → Fix Committed |
Revision history for this message
| 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 |
To post a comment you must log in.
Tentatively assigning to 2.6.0. We doing other UI work in this area so we can probably fix it at the same time.