Closed
Bug 789873
Opened 12 years ago
Closed 12 years ago
RequestWhiner doesn't respect SecureMail settings for "Outstanding Requests" mail
Categories
(bugzilla.mozilla.org :: Extensions, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: reed, Assigned: dkl)
Details
Attachments
(1 file)
3.51 KB,
patch
|
glob
:
review+
|
Details | Diff | Splinter Review |
The RequestWhiner extension doesn't respect SecureMail settings for the "Outstanding Requests" weekly mail that gets sent out.
we need to add an x-bugzilla-type header, and add code to securemail similar to whine processing.
Assignee | ||
Comment 3•12 years ago
|
||
This makes the request whines work similarly to standard whines. 1. The request whine header contains X-Bugzilla-Type: whine 2. SecureMail sees the header and then parses the text/plain part of the email for bug ids. 3. If one or more bug ids are secure bugs, it will encrypt the entire message with the user's key/cert or replace the body with a message stating it was secure. 4. I also updated the code in whineatrequests.pl to use proper building of multipart messages using Email::MIME similar to bug 714724. dkl
Attachment #659854 -
Flags: review?(glob)
Comment on attachment 659854 [details] [diff] [review] Patch to encrypt requestwhine emails if contains secure bugs (v1) r=glob
Attachment #659854 -
Flags: review?(glob) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.0 modified extensions/RequestWhiner/bin/whineatrequests.pl Committed revision 8324. Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.0 added extensions/RequestWhiner/template/en/default/requestwhiner/header.txt.tmpl Committed revision 8325. Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2 modified extensions/RequestWhiner/bin/whineatrequests.pl added extensions/RequestWhiner/template/en/default/requestwhiner/header.txt.tmpl Committed revision 8346.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: Extensions: Other → Extensions
You need to log in
before you can comment on or make changes to this bug.
Description
•