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://staging/~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) |
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 |
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.