Bug 458704 - Bugzilla no longer includes email headers it used to include
Summary: Bugzilla no longer includes email headers it used to include
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Email Notifications
Version: 3.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Lawrence
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-11 17:59 UTC by Doug Ledford
Modified: 2013-06-24 04:08 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-08-14 21:07:37 UTC
Embargoed:


Attachments (Terms of Use)
Patch to add additional headers to flag emails (v1) (1.21 KB, patch)
2008-08-11 22:45 UTC, David Lawrence
nelhawar: review+
Details | Diff

Description Doug Ledford 2008-08-11 17:59:22 UTC
Description of problem: I used to filter my bugzilla emails based upon a number of headers, one of which is the presence of X-(Loop|BeenThere|Sender):.*bugzilla (which I used to identify Red Hat bugzillas and which used to be present on all redhat bugzilla emails) and the other header I used was X-Bugzilla-Reason:

Now, some emails don't include these headers.  For instance, when I get an email telling me I got a QE-ack on my bug, and that I know have rhel-5.3.0 granted on my bug (the bugbot seeing all three acks in place), those notification emails are missing both of the above mentioned headers that they used to have.


Version-Release number of selected component (if applicable):


How reproducible: 100%


Steps to Reproduce:
1. Get an email from the bugzilla bot as it notices/sets flags, it will be absent some of the normal bugzilla headers.
2.
3.
  
Actual results:
My mail filters drop the emails into my Inbox

Expected results:
The mails should go into one of my various bugzilla folders.


Additional info:

Comment 1 David Lawrence 2008-08-11 22:45:10 UTC
Created attachment 314012 [details]
Patch to add additional headers to flag emails (v1)

Patch attached to add missing headers to flag emails.

Please review

Dave

Comment 2 Noura El hawary 2008-08-14 09:58:44 UTC
Comment on attachment 314012 [details]
Patch to add additional headers to flag emails (v1)

Hey Dave, Looks good to me , tested it and it works good.

Noura

Comment 3 David Lawrence 2008-08-14 21:07:37 UTC
Pushed to CVS and is now live on production.

Dave


Note You need to log in before you can comment on or make changes to this bug.