Db permits email addresses with no Person

Bug #335509 reported by Guilherme Salgado
0
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
William Grant

Bug Description

In the beginning, all email addresses had to be associated with a person, but recently this has changed, so we *can* have email addresses that are not associated to a Person. This has not been a problem until now, because we don't have any way of creating email addresses not associated with a person.

However, with the new SSO server, personless accounts will be created, thus creating email addresses without an associated person. And we have plenty of code which is not ready to cope with that (e.g. all callsites of PersonSet.getByEmail()), so we'll have to do something about it.

Changed in launchpad-registry:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Curtis Hovey (sinzui) wrote :

I do not understand why this is High priority. I do not think PersonSer should know about email addresses that belong to accounts. I expect user profiles to be created when the user registers with launchpad, or logs in for the first time using the SSO.

Curtis Hovey (sinzui)
visibility: private → public
Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: none → series-future
Revision history for this message
Curtis Hovey (sinzui) wrote :

This bug, if it is still legitmate is a foundation issue. Every account that launchpad knows about should has a person.

Changed in launchpad-registry:
milestone: series-future → none
affects: launchpad-registry → launchpad-foundations
Stuart Bishop (stub)
Changed in launchpad-foundations:
status: Triaged → Invalid
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Stuart, why is this Invalid?

Changed in launchpad-foundations:
status: Invalid → Triaged
Revision history for this message
Stuart Bishop (stub) wrote : Re: [Bug 335509] Re: Must deal with email addresses that have no associated Person

On Wed, Jun 23, 2010 at 10:16 PM, Francis J. Lacoste
<email address hidden> wrote:
> Stuart, why is this Invalid?

Because we no longer have any reason to store EmailAddress records
that are not linked to a Person.

Oh... except shipit. Bugger.

--
Stuart Bishop <email address hidden>
http://www.stuartbishop.net/

Revision history for this message
Robert Collins (lifeless) wrote : Re: Must deal with email addresses that have no associated Person

Shipit is gone. We can cleanup the related code and tables now.

summary: - Must deal with email addresses that have no associated Person
+ Db permits email addresses with no Person
William Grant (wgrant)
Changed in launchpad:
assignee: nobody → William Grant (wgrant)
status: Triaged → Fix Released
Curtis Hovey (sinzui)
tags: added: disclosure sharing
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.