OOPS when registering a new account on login.lp.net with an email address that belongs to an existing profile

Bug #361026 reported by Guilherme Salgado
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Guilherme Salgado

Bug Description

If someone tries to register using an email address that belongs to an existing profile which does not use Launchpad, we pretend we've not seen that email address before and let the user continue with the registration process. Then, at the end of the process we just change the necessary bits on that existing profile instead of creating a new one.

This doesn't work when you use the registration form on login.lp.net:
OOPS-1200S2282 -- ProgrammingError: permission denied for relation account
OOPS-1213S88 -- ProgrammingError: permission denied for relation person (on staging)

More ProgrammingError: permission denied for relation person: OOPS-1220I561, OOPS-1220J559, OOPS-1220J560, OOPS-1220J561, OOPS-1220I636, OOPS-1220J645 (all in lpnet), OOPS-1250J303, OOPS-1250I14

Changed in launchpad-foundations:
status: New → Triaged
tags: added: oops
description: updated
Ursula Junque (ursinha)
description: updated
Ursula Junque (ursinha)
description: updated
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

This makes the OOPS report daily.

Stuart can you see if there is an easy fix?

Changed in launchpad-foundations:
assignee: nobody → Stuart Bishop (stub)
importance: Undecided → High
Changed in launchpad-foundations:
milestone: none → 2.2.6
Revision history for this message
Stuart Bishop (stub) wrote :

There is no easy fix.

The authtoken methods (the token used for the SSO server) are trying to modify Person records. The SSO server cannot do this, as it needs to remain operational when the Launchpad database is down for maintenance.

The workflows need to be fixed so that the SSO registration forms only deal with Account, and anything that needs to deal with Person or Account+Person needs to redirect to the main Launchpad site.

Stuart Bishop (stub)
Changed in launchpad-foundations:
assignee: Stuart Bishop (stub) → nobody
milestone: 2.2.6 → 2.2.7
Curtis Hovey (sinzui)
Changed in launchpad-foundations:
assignee: nobody → Guilherme Salgado (salgado)
Revision history for this message
Guilherme Salgado (salgado) wrote :

I think there's an easy fix for this bug. We could detect when SSO users attempt to register using an email address that is already associated with a Launchpad profile and ask them to either claim that profile (so that the credentials can be used on SSO) or register with a different email address.

The other alternative I see is to redirect from login.lp.net to lp.net on the last step of the registration, so that the changes to the Person entry can be done. At that point we'd have to present a message explaining to the user why we're creating a LP account instead of an SSO one (as requested).

Changed in launchpad-foundations:
status: Triaged → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit

Fixed in devel r8799.

Changed in launchpad-foundations:
status: In Progress → Fix Committed
Revision history for this message
Curtis Hovey (sinzui) wrote : Bug 361026 Fix released

Fixed released in Launchpad sinzui.

Changed in launchpad-foundations:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.