1027638 - Use product/component in subject of secure bugs rather than duplicating the bug number

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Ed Morley [:emorley]]

Current secure bugmail has a subject of something like:
> [Bug 853396] (Secure bug 853396 updated)

Repeating the bug number doesn't seem ideal, and given that metadata such as product and component is already available in the message headers (and as such, we wouldn't be revealing anything more by including it in the summary), I think we should instead use:

> [Bug 853396] (Secure bug in Product::Component updated)
or
> [Bug 853396] (Secure Product::Component bug updated)

Or something along those lines.

Failing that, even "[Bug 853396] (Secure bug updated)" would be less redundant than the current subject.

I'm guessing it would need tweaking at:
https://github.com/mozilla/webtools-bmo-bugzilla/blob/a94a60e0eb2a9d213a7182136a881b77304ec7d1/extensions/SecureMail/Extension.pm#L548

Comment 1

[Ed Morley [:emorley]]

Attachment: Change to "Secure bug in $product::$component"

Previously the subject line of secure bugs would be something like:
[Bug 123456] (Secure bug 123456 updated)

With this patch, they are now:
[Bug 123456] (Secure bug in product::component)

Since:
* The product & component are already in plaintext in the message headers.
* No other subjects include the word "updated", and for new bugs we have until now appended "updated" in the summary, even though really they are "created". Happy to add this back in with a conditional on $add_new for updated/created if you would prefer.

Thanks :-)

Comment 2

[:glob ✱]

Comment on attachment 8442819 [details] [diff] [review]
Change to "Secure bug in $product::$component"

Review of attachment 8442819 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob

thanks for the patch ed :)

::: extensions/SecureMail/Extension.pm
@@ +551,5 @@
>          # the format of the subject line in another way.
>          my $new = $add_new ? ' New:' : '';
> +        my $product = $email->header('X-Bugzilla-Product');
> +        my $component = $email->header('X-Bugzilla-Component');
> +        $subject =~ s/($bug_id\])\s+(.*)$/$1$new (Secure bug in $product::$component)/;

"$product::" is a valid variable name in perl, so this is empty.

however it looks better with spaces before/after :: and fixes that issue (the alternative fix is ${product}::).

i'll make this change prior to committing for you.

Comment 3

[:glob ✱]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   ea5e2e6..d7b36fd  master -> master

Comment 4

[Ed Morley [:emorley]]

Great, thank you :-)

And yeah wasn't sure about Perl variables - should have guessed that would be a valid name haha.

(Not that it matters, but don't know if you noticed that your patch import/naming script uses the Bugzilla display name rather than the patch author set in the attachment - for this and bug 649691 the commit username had the [:foo] from the display name)

Comment 5

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Repeating the bug number is necessary for proper threading in gmail.  You've reintroduced bug 737883.  Please back this out.

Comment 6

[Reed Loden [:reed]]

(In reply to Kyle Huey [:khuey] ([e-mail]) from comment #5)
> Repeating the bug number is necessary for proper threading in gmail.  You've
> reintroduced bug 737883.  Please back this out.

or just add the bug id...

Comment 7

[:glob ✱]

(In reply to Kyle Huey [:khuey] ([e-mail]) from comment #5)
> Repeating the bug number is necessary for proper threading in gmail.  You've
> reintroduced bug 737883.  Please back this out.

bug 1030747

Comment 8

[Ed Morley [:emorley]]

Thank you Byron :-)

(And ffs gmail...)