You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "David (JIRA)" <ji...@apache.org> on 2014/11/12 00:41:34 UTC

[jira] [Comment Edited] (FILEUPLOAD-235) FileItem.getName() does not conform to the Java File.getName behaviour and *returns* a full path when provided with one

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

David edited comment on FILEUPLOAD-235 at 11/11/14 11:41 PM:
-------------------------------------------------------------

[~tn] is there a tracker for the next major version (e.g. 2.X) of commons fileload that I can raise this issue in perhaps ?


was (Author: dblack):
[~tn] is there a tracker for commons fileload 2.X branch that I can raise this issue in perhaps ?

> FileItem.getName() does not conform to the Java File.getName behaviour and *returns* a full path when provided with one
> -----------------------------------------------------------------------------------------------------------------------
>
>                 Key: FILEUPLOAD-235
>                 URL: https://issues.apache.org/jira/browse/FILEUPLOAD-235
>             Project: Commons FileUpload
>          Issue Type: Bug
>    Affects Versions: 1.3
>            Reporter: David
>
> I understand that FileItem is simply following RFC 1867, but the implementation of 'getName' 
> method on a file related class IMHO should follow the behaviour of the java File class which
> bq. Returns the name of the file or directory denoted by this abstract pathname. This is just the last name in the pathname's name sequence. If the pathname's name sequence is empty, then the empty string is returned. http://docs.oracle.com/javase/7/docs/api/java/io/File.html#getName()
> I propose that FileItem.getName() is changed to *match* the java File class behaviour and that another method called something like 'getRawName' be added to the FileItem class. The rational for this change is to make it less easy to shoot oneself in the foot (in no modern web application is a 'full filepath' useful, especially when most browsers do not send one!).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)