509863 - [MODULE] should linkify crash IDs to crash stats

Status: RESOLVED FIXED

(Webtools Graveyard :: Mozbot, defect)

Description

[[:Cww]]

Attachment: Crashstats.bm

We've been using this module a LOT on sumobot to help our contributors quickly linkify crash Ids to crash-stats.  Time to share!

The algorithm is completely dumb and just linkifies anything that looks like it'd match.

Comment 1

[Tanner M. Young [:tmyoung]]

I really like how this looks.  I can't wait see it in the QA channel with firebot.  Well done Cww, as usual.

Comment 3

[Tanner M. Young [:tmyoung]]

Added target milestone and other things from Bug 515780.

Comment 4

[Wolf [:wolf]]

Enabled in firebot for testing. :-) I'm slow.. heh

Comment 5

[Wolf [:wolf]]

Comment on attachment 393940 [details]
Crashstats.bm

Hrm.. It would be nice if this module included some kind of help text. Otherwise, it seems good. :-)

Comment 6

[Jesse Ruderman]

Please change the negative lookbehind assertion so it matches both http and https URLs.

<timeless> https://crash-stats.mozilla.com/report/index/d38d7c4d-353f-425a-b8a9-c5fb22100226
<firebot> timeless: http://crash-stats.mozilla.com/report/index/d38d7c4d-353f-425a-b8a9-c5fb22100226
<timeless> thanks dumb bot

Comment 7

[Tanner M. Young [:tmyoung]]

(In reply to comment #6)
> Please change the negative lookbehind assertion so it matches both http and
> https URLs.
> 
> <timeless>
> https://crash-stats.mozilla.com/report/index/d38d7c4d-353f-425a-b8a9-c5fb22100226
> <firebot> timeless:
> http://crash-stats.mozilla.com/report/index/d38d7c4d-353f-425a-b8a9-c5fb22100226
> <timeless> thanks dumb bot

If this hasn't been fixed in this patch, I will quickly write a patch, once this is committed in the repository.

Comment 8

[Wolf [:wolf]]

Checking in Crashstats.bm;
/cvsroot/mozilla/webtools/mozbot/BotModules/Crashstats.bm,v  <--  Crashstats.bm
initial revision: 1.1
done

Comment 9

[Tanner M. Young [:tmyoung]]

(In reply to comment #7)
> If this hasn't been fixed in this patch, I will quickly write a patch, once
> this is committed in the repository.

I have filed and submitted a patch for this in Bug 557012.