277209 - [XMLLogger] URLs should be flagged.

Status: RESOLVED INACTIVE

(Webtools Graveyard :: Mozbot, enhancement, P4)

Description

[Scott Venier]

The XMLLogger module should have configurable regular expressions that it uses
to flag messages with customizable xml tags.

More specifically, I set the variable flagPatterns to
'[a-z]+://[a-z0-9-]+(\.[a-z0-9-]+)+(/[^ "]*)?'=>"link" and every URL found gets
wrapped with <link> tags.

A possible patch will be attached shortly.

Comment 1

[Scott Venier]

Attachment: patch to add URL flagging support to mozbot.

This patch also puts a reference to a stylesheet into the XML output.  Sorry to
include both in a single patch.  I'll submit a second bug and attch the
stylesheets to it.

Comment 2

[Scott Venier]

I just discovered a bug in my patch...  around line 130 the if($message =~
m/(.*?)($pattern)(.*)/) and the line after it should just be:
$message =~ s|($pattern)|<$tag>$1</$tag>|g;

Much simpler, and works correctly.  Unfortunately, I don't have a corresponding
source file to generate a new patch.

Comment 3

[Wolf [:wolf]]

Resetting to new default owner.

Comment 4

[:glob ✱]

Mozbot/Firebot is in maintenance-only mode; no further changes are planned.