No way to get launchpad user id from a bug comment email

Bug #605340 reported by Steve Beattie
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Brian Murray

Bug Description

The last launchpad rollout fixed bug 111147, no longer exposing the email address of people who have hidden them in the launchpad interface. The unfortunate side effect of this is that there's now no programmatic way to go from an emailed comment back to the either the identity of the commenter or the specific comment on the bug report.

To give some context, I prepare a weekly thank you report of people who test out proposed fixes in Ubuntu. I give this report in both an irc and wiki context (for an example wiki report, see https://wiki.ubuntu.com/QATeam/SRUReports/2010-07-07), and I attempt to use the registered irc nicknames and wiki names when doing so.

Because the reported testing comments are free-form and are not identifiable programmatically, I do this by manually sifting through comments on the associated bug reports. For a week's report, this can involve examining a couple of hundred bug reports, thus it's far less time consuming to examine unread comments in an email client rather than opening up a couple of hundred bug web pages and scrolling to view the most recent comment (we have a specific team subscribed to those reports where a fix is proposed, and I filter email sent to that team). This workflow was possible because the commenter's email address was given as the From: address, and from that their irc nick and wiki name (if registered) could be looked up via the launchpad.people.findPerson(text=email) launchpadlib api, even for people who's email addresses were hidden in launchpad. (In hindsight, I see I should have been using launchpad.people.findByEmail().) None of the many X-Launchpad headers in an email message allow you to figure out the launchpad id of the person sending the comment.

I think protecting people's email addresses is important, particularly when we've told them that it is hidden, so I'm not asking that the fix for bug 111147 be reverted. Rather, I'd like to see a header added that (a) either gives the launchpad id of the commenter directly (note that *all* the commenters on a bug are already included in the X-Launchpad-Bug-Commenters: header, but it doesn't distinguish who *this* commenter is) or (b) identify which comment number the email is associated with, so that the launchpad id can be lookup from the specific bug.message's owner field.

Related branches

Revision history for this message
Graham Binns (gmb) wrote :

I'm working on fixing notifications at the moment. One way that we can fix your problem is with an X-Launchpad-*something (we haven't decided what it will be yet) header that specifies the person who triggered the notification. I'll try to get a branch in that adds that for you this week.

tags: added: story-better-bug-notification
Changed in malone:
status: New → Triaged
importance: Undecided → High
Graham Binns (gmb)
Changed in malone:
status: Triaged → In Progress
assignee: nobody → Graham Binns (gmb)
milestone: none → 10.08
Changed in malone:
assignee: Graham Binns (gmb) → nobody
status: In Progress → Triaged
Changed in malone:
assignee: nobody → Brian Murray (brian-murray)
status: Triaged → In Progress
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit
tags: added: qa-needstesting
Changed in malone:
status: In Progress → Fix Committed
Revision history for this message
Brian Murray (brian-murray) wrote :

This seems to be only partially fixed in that on staging only modifying the assignee of the bug task contained the X-Launchpad-Bug-Modifier header. We are going to land this as is and then create another branch to ensure that the X-Launchpad-Bug-Modifier shows up in all bug messages.

tags: added: qa-ok
removed: qa-needstesting
Changed in malone:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
removed: qa-ok
Ursula Junque (ursinha)
Changed in malone:
status: Fix Released → Fix Committed
milestone: 10.08 → 10.09
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Brian Murray (brian-murray) wrote :

This landed in a cherry pick on production so I'm changing the status to Fix Released.

14:22:33 - flash:[...l/mailing-lists/ubuntu-bugs] grep "X-Launchpad-Bug-Modifier" 2010-08
X-Launchpad-Bug-Modifier: Vladimir Rutsky (rutsky)
X-Launchpad-Bug-Modifier: Vladimir Rutsky (rutsky)
X-Launchpad-Bug-Modifier: Vladimir Rutsky (rutsky)
X-Launchpad-Bug-Modifier: Vladimir Rutsky (rutsky)
X-Launchpad-Bug-Modifier: Vladimir Rutsky (rutsky)
X-Launchpad-Bug-Modifier: Brian E. GEORGE (bredgeo1)
X-Launchpad-Bug-Modifier: Brian E. GEORGE (bredgeo1)
X-Launchpad-Bug-Modifier: Julien Lavergne (gilir)
X-Launchpad-Bug-Modifier: Ronald McCollam (fader)

Changed in malone:
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.