423412 - linkify report format from crash-reports

Status: RESOLVED FIXED

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

Description

[timeless]

if a user pastes an incident id, it will be of this form:
bp-de11b4fd-f3e6-11dc-8b25-001a4bd43ed6 

if an analyst copies a portion of the report from the server it will be of this form:
UUID    de11b4fd-f3e6-11dc-8b25-001a4bd43ed6

bugzilla has always linkified both equivalents for talkback :(

Comment 1

[Dave Miller [:justdave]]

Fortunately, talkback reports were uniquely identifiable because they were a specific format (specific number of letters on each end with digits in the middle, always a certain length).  Breakpad IDs, without the bp- in front, are not only not uniquely identifiable as breakpad IDs, but there are other known uses for identifier number in the exact same format:

  Extension IDs within Firefox
  Registry keys in Windows
  Device UUID identifiers in Linux

And thats just a few...

Comment 2

[timeless]

i meant that the form should be:
/^UUID\s+:uuid:$/

not just /:uuid:/

Comment 3

[Reed Loden [:reed]]

Changed to: \b(?:UUID\t|bp\-)([a-f0-9]{8}\-[a-f0-9]{4}\-[a-f0-9]{4}\-[a-f0-9]{4}\-[a-f0-9]{12})\b

Will go live during next update from prodpatches.

Comment 4

[timeless]

fwiw, copying w/ ie8b1 on my laptop yields this:
UUID de11b4fd-f3e6-11dc-8b25-001a4bd43ed6 

the paste in c0 (which has 4 spacs) was from the same incident id, you'll need to use:
\s+ instead of \t, sorry.

note that talkback has two formats:
TB41575058X
Incident ID: 41575058
The latter is the lesser known format but is the one similar to the UUID format that I was asking to have linkified.

Comment 5

[Reed Loden [:reed]]

My change went live. It's \t still, but if we start to see reports where \s+ would be better, I'll change it.