[ 
https://issues.apache.org/jira/browse/HADOOP-7531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13082709#comment-13082709
 ] 

Todd Lipcon commented on HADOOP-7531:
-------------------------------------

Another small thing: in getDecodedPath and getRawPath, I think you should do a 
Preconditions.checkArgument(request.getRequestURI().startsWith(servletName)). 
Otherwise the substring might chop something else off if the function is 
misused.

> Add servlet util methods for handling paths in requests 
> --------------------------------------------------------
>
>                 Key: HADOOP-7531
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7531
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: util
>    Affects Versions: 0.23.0
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>             Fix For: 0.23.0
>
>         Attachments: hadoop-7531-1.patch, hadoop-7531-1.patch, 
> hadoop-7531-2.patch
>
>
> Common side of HDFS-2235.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to