Emails entered in SSO are not available in LP

Bug #525930 reported by Roman Yepishev
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Canonical SSO provider
Fix Released
High
Danny Tamez

Bug Description

I have added my @canonical.com and @ubuntu.com addresses to SSO but then they did not show up in launchpad, therefore I was unable to send email to mailing lists from @canonical.com addresses.

So I added my @canonical.com address to lp and it broke :)

The @canonial.com/ubuntu.com emails are listed in the dropdown only, they are not mentioned as 'not verified' or similar.

I was advised to try creating new lp account with one of these emails and then merge my lp accounts but lp oopses on this attempt as well.

<stuartm> odd. I've just added two emails (unverified) to my sso account and they're already showing up in login.launchpad.net, but they're not visible in launchpad itself I suspect they've changed the way they handle their email states.

Changed in canonical-identity-provider:
status: New → Incomplete
assignee: nobody → Stuart Metcalfe (stuartmetcalfe)
importance: Undecided → High
Revision history for this message
Stuart Metcalfe (stuartmetcalfe) wrote :

Thanks for the bug report. There seems to be some inconsistency in the ways login.u.c, login.lp and lp itself handle emails. Here's an example:

I added an unverified email address to my login.u.c account

 * login.u.c: visible
 * login.lp: visible
 * lp itself: not visible

I deleted the email from login.u.c (which actually removes the data from the emailaddress table):

 * login.u.c: not visible
 * login.lp: visible
 * lp itself: not visible

There's obviously something not quite right here. It may be that the email remaining visible in login.lp is due to caching or replication lag. The emails never being visible in lp itself may also be caused by one of these but this obviously requires further investigation. We'll need to install all 3 apps locally and run a comprehensive test of all cases to identify the most likely source of these problems.

Revision history for this message
Guilherme Salgado (salgado) wrote :

Emails created by the code that serves login.u.c don't get their EmailAddress.person set. That's why they don't show up in Launchpad.

To make matters worse, if you set that newly created email as your preferred one, your LP profile will become invalid because it will have no preferred email.

Changed in canonical-identity-provider:
milestone: none → 2.3.0
Changed in canonical-identity-provider:
status: Incomplete → Confirmed
assignee: Stuart Metcalfe (stuartmetcalfe) → nobody
Changed in canonical-identity-provider:
milestone: 2.3.0 → 2.2.2
Danny Tamez (zematynnad)
Changed in canonical-identity-provider:
assignee: nobody → Danny Tamez (zematynnad)
assignee: Danny Tamez (zematynnad) → nobody
Changed in canonical-identity-provider:
milestone: 2.2.2 → none
Changed in canonical-identity-provider:
milestone: none → 2.2.2
assignee: nobody → Danny Tamez (zematynnad)
Revision history for this message
Danny Tamez (zematynnad) wrote :

2 scripts are also needed:
One to delete the duplicate emails with as little impact on existing users.
Another to hook up entries in the lp_person table with the accounts created in SSO.

Revision history for this message
Guilherme Salgado (salgado) wrote : Re: [Bug 525930] Re: Emails entered in SSO are not available in LP

On Fri, 2010-03-05 at 19:27 +0000, Danny Tamez wrote:
> 2 scripts are also needed:
> One to delete the duplicate emails with as little impact on existing users.

This one is for a different bug, no? That one about mixed-case email
addresses...

> Another to hook up entries in the lp_person table with the accounts created in SSO.

I have one which does the latter already. It's at
https://wiki.canonical.com/InformationInfrastructure/OSA/LaunchpadProductionStatus but hasn't been executed yet.

--
Guilherme Salgado <email address hidden>

Revision history for this message
Danny Tamez (zematynnad) wrote :

Thanks Guilherme - I've added a comment for the first script mentioned above to bug #523346

Danny Tamez (zematynnad)
Changed in canonical-identity-provider:
status: Confirmed → Fix Committed
Revision history for this message
Julien Funk (jaboing) wrote :

Can this be tested prior to staging? If so, how?

Julien Funk (jaboing)
Changed in canonical-isd-qa:
assignee: nobody → Julien Funk (jaboing)
Revision history for this message
Guilherme Salgado (salgado) wrote :

I think you'd have to query the database for a newly created email
address and verify that EmailAddress.person is set -- currently it
isn't.

Revision history for this message
Dave Morley (davmor2) wrote :

Needs to be tested on staging in order to tie into lp staging to test for the cross platform breakage.

Changed in canonical-isd-qa:
assignee: Julien Funk (jaboing) → Dave Morley (davmor2)
Revision history for this message
Danny Tamez (zematynnad) wrote :

Steps to reproduce/test:
1. Go to http://www.guerrillamail.com/ and setup a temporary email address
2. Go to https://login.staging.ubuntu.com and log in with your canonical email address and password
3. Click on Add another email and enter your temporary guerillamail email address.
4 Click on Continue
5. Reload the http://www.guerrillamail.com/ page to get your verification email.
6. Click on the link to verify.
7. You'll see that the email has been verified.
8. Log in to https://login.staging.launchpad.net with your canonical email address and password
9. Expected behavior: Your guerilla.com temp email address is listed under your emails.

Revision history for this message
Dave Morley (davmor2) wrote :

This mail shows up in staging correctly.

Changed in canonical-isd-qa:
status: New → Fix Committed
Revision history for this message
Dave Morley (davmor2) wrote :

Continuing on from the adding an email account to an existing registered account.

The new account shows up in launchpad correctly also.

Changed in canonical-isd-qa:
status: Fix Committed → In Progress
status: In Progress → Fix Released
Revision history for this message
Roman Yepishev (rye) wrote :

Confirming that all my SSO E-Mails are now available in LP.
Thanks!

Revision history for this message
Danny Tamez (zematynnad) wrote :

Cleanup script has been run in production successfully.

Danny Tamez (zematynnad)
Changed in canonical-identity-provider:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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