Closed Bug 292886 Opened 20 years ago Closed 19 years ago

X-Bugzilla-Product and X-Bugzilla-Component headers should exist by default

Categories

(Bugzilla :: Email Notifications, enhancement)

2.19.2
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: mkanat, Assigned: Wurblzap)

Details

Attachments

(1 file)

My new mozbot-receives-bugmail-and-reports-it-to-IRC code depends on the
X-Bugzilla-Product and X-Bugzilla-Component headers. I noticed that my bot
wasn't reporting flag changes, and when I investigated, I discovered that at
least from b.m.o, the X-Bugzilla-Product and X-Bugzilla-Component headers aren't
being set in flag mail, at least from bug flags.
I'd really like to get this fix into 2.18, or my mozbot code will be pretty useless.
Target Milestone: --- → Bugzilla 2.18
OK, I should be more specific -- the headers are actually missing from my
*bugmail*, not the "request" mail.
Summary: X-Bugzilla-Product and X-Bugzilla-Component headers are missing from flag mail → X-Bugzilla-Product and X-Bugzilla-Component headers are missing from bugmail when the only change is a flag
Arrgh, the actual problem is that Bugzillas don't contain these headers by
default. Adding them by default wouldn't be too hard.
Severity: normal → enhancement
Summary: X-Bugzilla-Product and X-Bugzilla-Component headers are missing from bugmail when the only change is a flag → X-Bugzilla-Product and X-Bugzilla-Component headers should exist by default
Target Milestone: Bugzilla 2.18 → ---
Version: 2.18 → 2.19.2
Assignee: attach-and-request → email-notifications
Component: Attachments & Requests → Email Notifications
Isn't this bug #70710?
(In reply to comment #4)
> Isn't this bug #70710?

  No -- we already have %component% and %product%, they're just not being used
by default.
Attached patch PatchSplinter Review
Assignee: email-notifications → wurblzap
Status: NEW → ASSIGNED
Attachment #192338 - Flags: review?(mkanat)
Comment on attachment 192338 [details] [diff] [review]
Patch

This makes perfect sense to me. At least, it's easier to remove them if we
don't want them.
r=LpSolit for 2.20 and tip only.
Attachment #192338 - Flags: review+
Attachment #192338 - Flags: review?(mkanat)
Flags: approval?
Flags: approval2.20?
Target Milestone: --- → Bugzilla 2.20
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
tip:

Checking in defparams.pl;
/cvsroot/mozilla/webtools/bugzilla/defparams.pl,v  <--  defparams.pl
new revision: 1.166; previous revision: 1.165
done

2.20rc2:

Checking in defparams.pl;
/cvsroot/mozilla/webtools/bugzilla/defparams.pl,v  <--  defparams.pl
new revision: 1.160.2.2; previous revision: 1.160.2.1
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: