[
https://issues.apache.org/jira/browse/CHUKWA-642?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13469198#comment-13469198
]
Eric Yang commented on CHUKWA-642:
----------------------------------
Could you add a test case? Thanks
> Unvalidated Regular Expression Usage
> ------------------------------------
>
> Key: CHUKWA-642
> URL: https://issues.apache.org/jira/browse/CHUKWA-642
> 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
> ([http://types.cs.washington.edu/checker-framework/current/checkers-manual.html#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: http://www.atlassian.com/software/jira
