Oops updating +contactaddress of a team with mailing list
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Critical
|
Barry Warsaw |
Bug Description
As seen in OOPS-818B1006, somehow the +contactaddress page of a team with a mailing list tried to set a None email address as the preferred team's email.
The error is: TypeError: Any person's email address must provide the IEmailAddress interface. None doesn't.
Relevant part of the chat I had with barry about this issue:
<matsubara> somehow the mailing_
<barry> matsubara: i wonder if that's related to the config fix i'm pushing through pqm now
<matsubara> actually the mailing_
<matsubara> barry: what's it?
<barry> matsubara: https:/
<barry> matsubara: i just realized that this might be a more severe problem
<barry> matsubara: if a mailing list was approved using edge's bogus hostname on one machine, and then we try to access that on a different machine, you'll get that None
<barry> and i approved a bunch of new mailing lists this morning
<barry> damn. i'm going to try to put together a query to at least find those bogus email addresses
Changed in launchpad: | |
assignee: | nobody → barry |
milestone: | none → 1.2.3 |
status: | New → Confirmed |
Changed in launchpad: | |
status: | In Progress → Fix Released |
A fix for this is winding its way through PQM and will get cherrypicked as soon as it lands. We'll need to back-patch the email addresses for about 8 mailing lists that I approved this morning while the bug still existed, but I have a script to do that and we'll apply that soon too.