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)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: mkanat, Assigned: Wurblzap)
Details
Attachments
(1 file)
|
523 bytes,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
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.
| Reporter | ||
Comment 1•20 years ago
|
||
I'd really like to get this fix into 2.18, or my mozbot code will be pretty useless.
Target Milestone: --- → Bugzilla 2.18
| Reporter | ||
Comment 2•20 years ago
|
||
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
| Reporter | ||
Comment 3•20 years ago
|
||
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 → ---
| Reporter | ||
Updated•20 years ago
|
Version: 2.18 → 2.19.2
| Reporter | ||
Updated•20 years ago
|
Assignee: attach-and-request → email-notifications
Component: Attachments & Requests → Email Notifications
Comment 4•20 years ago
|
||
Isn't this bug #70710?
| Reporter | ||
Comment 5•20 years ago
|
||
(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.
| Assignee | ||
Comment 6•19 years ago
|
||
Assignee: email-notifications → wurblzap
Status: NEW → ASSIGNED
Attachment #192338 -
Flags: review?(mkanat)
Comment 7•19 years ago
|
||
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+
| Assignee | ||
Updated•19 years ago
|
Attachment #192338 -
Flags: review?(mkanat)
| Assignee | ||
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
| Assignee | ||
Updated•19 years ago
|
Target Milestone: --- → Bugzilla 2.20
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
Comment 8•19 years ago
|
||
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.
Description
•