Closed
Bug 1190476
Opened 9 years ago
Closed 9 years ago
set Comment field in GPG email to the URL of the bug
Categories
(bugzilla.mozilla.org :: Extensions, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Atoll, Unassigned)
Details
Attachments
(1 file)
4.83 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
When sending bugmail, we plaintext reference the bug # in the email plaintext headers. It would be helpful to have a direct link to the bug in the plaintext email as well (when viewing GPG emails on non-GPG clients). Could you please activate the 'comment' attribute in the Crypt::OpenGPG code? https://github.com/btrott/Crypt-OpenPGP/blob/77352f28bfa9f9ad9f31f8ac145b0be39c5addba/lib/Crypt/OpenPGP/Armour.pm#L142 indicates that you just need to add a 'Comment => $bmo_url' parameter to the Headers hash. I request that the URL is the *only thing* in the Comment, since no further information is required and that provides the best usability (obvious clickable url on line 4 of the email).
Comment 1•9 years ago
|
||
Weird. I thought of this idea just the other day, then forgot it again :-) Gerv
i performed some testing with atoll and this appears to do the trick.
Attachment #8649705 -
Flags: review?(dkl)
Before: -----BEGIN PGP MESSAGE----- Version: Crypt::OpenPGP 1.04 hQIOA7Z/9NBBpcqLEAf/bMV+r9xTcPKkbUmoxzjFs8NlVQ/v4vyeAVF9AhTr70JP After: -----BEGIN PGP MESSAGE----- Comment: http://bz/1190476/show_bug.cgi?id=888 hQMMA7Z/9NBBpcqLEAgAilS8GOEtWVvdLlFTxU1jd+8rH3zxiJUbIZbiTgTMXeBM The message decrypts cleanly and to the same content as before.
Comment 4•9 years ago
|
||
Comment on attachment 8649705 [details] [diff] [review] 1190476_1.patch Review of attachment 8649705 [details] [diff] [review]: ----------------------------------------------------------------- r=dkl
Attachment #8649705 -
Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git df1f6c9..667ecc3 master -> master
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Assignee: glob → nobody
Component: Extensions: SecureMail → Extensions
You need to log in
before you can comment on or make changes to this bug.
Description
•