accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kyle Van Gilson (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-4677) Sanitize @PathParam and @QueryParam parameters in new REST-based monitor
Date Thu, 03 Aug 2017 23:52:01 GMT


Kyle Van Gilson commented on ACCUMULO-4677:

Saw this and took a swing at it in github PR #289.  If your interested, let me know as it
probably needs additional unit tests and tighter validation on bounds etc.  If you were thinking
along different lines, feel free to decline.  Probably best to look at the ParameterValidator
class before looking at the rest as it contains the meat of the validation.  Regards!

> Sanitize @PathParam and @QueryParam parameters in new REST-based monitor
> ------------------------------------------------------------------------
>                 Key: ACCUMULO-4677
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: monitor
>            Reporter: Christopher Tubbs
>            Assignee: Mike Walch
>            Priority: Blocker
>             Fix For: 2.0.0
>          Time Spent: 10m
>  Remaining Estimate: 0h
> Following on the issue identified in ACCUMULO-4660, I verified that parameters to the
REST-based monitor (ACCUMULO-3005) resources need sanitization as well.
> All {{@PathParam}} and {{@QueryParam}} annotated fields should be sanitized.

This message was sent by Atlassian JIRA

View raw message