No way to get launchpad user id from a bug comment email
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:/
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.
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-
Related branches
- Jelmer Vernooij (community): Approve (code)
-
Diff: 68 lines (+15/-5)3 files modifiedlib/canonical/launchpad/mailnotification.py (+1/-1)
lib/lp/bugs/doc/bugnotification-email.txt (+3/-1)
lib/lp/bugs/mail/bugnotificationbuilder.py (+11/-3)
- Māris Fogels (community): Approve
-
Diff: 77 lines (+61/-1)2 files modifiedlib/lp/bugs/mail/tests/test_bug_task_modification.py (+60/-0)
lib/lp/bugs/scripts/bugnotification.py (+1/-1)
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 |
Changed in malone: | |
status: | Fix Committed → Fix Released |
Changed in malone: | |
status: | Fix Released → Fix Committed |
milestone: | 10.08 → 10.09 |
tags: |
added: qa-ok removed: qa-needstesting |
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.