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

Ayush Saxena commented on HDFS-16503:
-------------------------------------

Committed to trunk.
Thanx [~tomscut] for the contribution!!!

> Should verify whether the path name is valid in the WebHDFS
> -----------------------------------------------------------
>
>                 Key: HDFS-16503
>                 URL: https://issues.apache.org/jira/browse/HDFS-16503
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: tomscut
>            Assignee: tomscut
>            Priority: Major
>              Labels: pull-request-available
>         Attachments: image-2022-03-14-09-35-49-860.png
>
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
>
> When creating a file using WebHDFS, there are two main steps:
> 1. Obtain the location of the Datanode to be written.
> 2. Put the file to this location.
> Currently *NameNodeRpcServer* verifies that pathName is valid, but 
> *NamenodeWebHdfsMethods* and *RouterWebHdfsMethods* do not.
> So if we use an invalid path(such as duplicated slash), the first step 
> returns success, but the second step throws an {*}InvalidPathException{*}. 
> IMO, we should also do the validation in WebHdfs, which is consistent with 
> the NameNodeRpcServer.
> !image-2022-03-14-09-35-49-860.png|width=548,height=164!
> The same webHDFS operations are: CREATE, APPEND, OPEN, GETFILECHECKSUM. So we 
> can add DFSUtil.isValidName to redirectURI for *NamenodeWebHdfsMethods* and 
> *RouterWebHdfsMethods.*



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to