Export OpenIdIdentifier to Canonical SSO
Bug #644975 reported by
Stuart Bishop
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical SSO provider |
Fix Released
|
Critical
|
Łukasz Czyżykowski | ||
| Launchpad itself |
Fix Released
|
High
|
Stuart Bishop | ||
Bug Description
Currently, in the information we export to the Canonical SSO we can only link a Person to a single OpenIdIdentifier. We use the last OpenIdIdentifier that was used to log into Launchpad, This can still cause confusion such as Bug #644824 but is the best we can do with the current data model.
We should export the OpenIdIdentifier table to the Canonical SSO. I expect using the same mechanism is best for now - create a lp_OpenIdIdentifier mirror table and replicate that via Slony-I.
The Canonical SSO should use this information to map SSO accounts to Launchpad People instead of using lp_Account.
We can then drop lp_Account from replication, and drop the table.
Related branches
lp://staging/~canonical-isd-hackers/canonical-identity-provider/644975-openididentifier
- Anthony Lenton (community): Approve
-
Diff: 656 lines (+106/-122)15 files modifiedidentityprovider/fixtures/test.json (+13/-37)
identityprovider/management/commands/populate.py (+10/-8)
identityprovider/models/account.py (+26/-19)
identityprovider/models/person.py (+13/-9)
identityprovider/tests/test_admin.py (+5/-3)
identityprovider/tests/test_command_cleanup.py (+1/-1)
identityprovider/tests/test_command_populate.py (+4/-4)
identityprovider/tests/test_middleware.py (+5/-5)
identityprovider/tests/test_models_account.py (+3/-8)
identityprovider/tests/test_models_person.py (+6/-4)
identityprovider/tests/test_utils.py (+3/-10)
identityprovider/tests/test_views_server.py (+9/-6)
identityprovider/tests/test_widgets.py (+5/-4)
identityprovider/tests/utils.py (+1/-1)
identityprovider/webservice/models.py (+2/-3)
| Changed in launchpad-foundations: | |
| status: | New → Triaged |
| Changed in launchpad-foundations: | |
| importance: | Undecided → High |
| assignee: | nobody → Stuart Bishop (stub) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #1 |
| Changed in launchpad-foundations: | |
| milestone: | none → 10.10 |
| tags: | added: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Triaged → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Stuart Metcalfe (stuartmetcalfe) wrote | #3 |
| Changed in canonical-identity-provider: | |
| importance: | Undecided → Critical |
| status: | New → Incomplete |
Revision history for this message
| Stuart Bishop (stub) wrote | #4 |
| Changed in canonical-identity-provider: | |
| status: | Incomplete → Confirmed |
Revision history for this message
| Stuart Metcalfe (stuartmetcalfe) wrote | #5 |
| Changed in canonical-identity-provider: | |
| status: | Confirmed → Incomplete |
Revision history for this message
| Stuart Metcalfe (stuartmetcalfe) wrote | #6 |
| Changed in canonical-identity-provider: | |
| status: | Incomplete → Confirmed |
| milestone: | none → 1-commitment |
| tags: | added: defect |
| Changed in canonical-identity-provider: | |
| assignee: | nobody → Łukasz Czyżykowski (lukasz-czyzykowski) |
| status: | Confirmed → In Progress |
| Changed in canonical-identity-provider: | |
| milestone: | 1-commitment → 2-implementation |
| Changed in canonical-identity-provider: | |
| importance: | Critical → High |
| tags: | added: kb-defect |
| tags: |
added: kb-task removed: kb-defect |
| Changed in canonical-identity-provider: | |
| importance: | High → Critical |
| tags: |
added: kb-defect removed: kb-task |
| Changed in canonical-identity-provider: | |
| status: | In Progress → Fix Committed |
| milestone: | 2-implementation → 3-internal-qa |
Revision history for this message
| Julien Funk (jaboing) wrote | #7 |
| Changed in canonical-identity-provider: | |
| milestone: | 3-internal-qa → 11.02 |
| Changed in canonical-identity-provider: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in db-stable r9862 <http://