chukwa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Spishak (JIRA)" <>
Subject [jira] [Updated] (CHUKWA-642) Unvalidated Regular Expression Usage
Date Wed, 26 Sep 2012 02:12:07 GMT


Eric Spishak updated CHUKWA-642:

    Attachment: CHUKWA-642-1.patch
> Unvalidated Regular Expression Usage
> ------------------------------------
>                 Key: CHUKWA-642
>                 URL:
>             Project: Chukwa
>          Issue Type: Bug
>            Reporter: Eric Spishak
>            Assignee: Eric Spishak
>         Attachments: CHUKWA-642-1.patch, CHUKWA-642.patch
> There are seven additional places in Chukwa where regular expressions are used without
first being validated as syntactically valid regular expressions. These could lead to unhelpful
PatternSyntaxException strack traces instead of useful error messages. Unfortunately, I was
not able to produce test conditions to highlight any of these issues.
> The attached patch fixes the issues. Note that the patch adds a small RegexUtil class
with helper methods to determine whether a String is a valid regular expression and to generate
error messages for invalid regular expressions. I feel that these helper methods are more
readable than catching the PatternSyntaxException. Furthermore, they allow simpler re-use
if needed elsewhere.
> I discovered these bugs using a tool named the Regex Checker ([]).
If you periodically run it on your codebase, then you will learn of other errors like this
as soon as they appear.  If you are interested, I can share my annotations for Chukwa, which
will enable you to run the Regex Checker yourself without any additional effort.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message