Success page should not be shown if ping failed
Bug #667893 reported by
Natalia Bidart
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Single Sign On Client |
Fix Released
|
High
|
Natalia Bidart | ||
| ubuntu-sso-client (Ubuntu) |
Fix Released
|
High
|
Natalia Bidart | ||
Bug Description
Right now, success page is being shown when login/registration is successful, and the ping is being made after that fact. So the ping may fail and a error message may be shown to the user even if a previous success messages was displayed!
Related branches
lp://staging/~nataliabidart/ubuntu-sso-client/delay-success-after-ping
- Roman Yepishev (community): Approve (fieldtest)
- Alejandro J. Cura (community): Approve
-
Diff: 961 lines (+339/-150)5 files modifieddata/ui.glade (+75/-53)
ubuntu_sso/credentials.py (+40/-16)
ubuntu_sso/gui.py (+59/-43)
ubuntu_sso/tests/test_credentials.py (+89/-9)
ubuntu_sso/tests/test_gui.py (+76/-29)
| Changed in ubuntu-sso-client: | |
| status: | New → Triaged |
| assignee: | nobody → Naty Bidart (nataliabidart) |
| importance: | Undecided → High |
| tags: | added: desktop+ u1-natty |
| Changed in ubuntu-sso-client: | |
| status: | Triaged → Fix Committed |
| Changed in ubuntu-sso-client (Ubuntu): | |
| status: | New → Triaged |
| importance: | Undecided → High |
| assignee: | nobody → Naty Bidart (nataliabidart) |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #1 |
| Changed in ubuntu-sso-client (Ubuntu): | |
| status: | Triaged → Fix Released |
| Changed in ubuntu-sso-client: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This bug was fixed in the package ubuntu-sso-client - 1.1.2-0ubuntu1
---------------
ubuntu-sso-client (1.1.2-0ubuntu1) natty; urgency=low
* New upstream release:
[ Natalia B. Bidart <email address hidden> ]
* The success page is not shown until the backend notifies that the ping
finished successfully (LP: #667893).
* Added a new DBus signal UserNotValidated to indicate when a user is
registered but not validated (LP: #667899).
* Added new workflow so email validation is requested if necessary.
-- Natalia Bidart (nessita) <email address hidden> Thu, 04 Nov 2010 11:00:45 -0300