DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21321>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21321

new methods: getFilePathName() and setInputStream()

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID



------- Additional Comments From [EMAIL PROTECTED]  2003-10-22 05:04 -------
The getFileName() method you are referring to is part of the Struts FormFile 
interface, and not part of Commons FileUpload. FileUpload makes available 
whatever the browser provides; Struts provides only the base file name.

It would be possible to add a method to the Struts FormFile interface to access 
the original file name, as provided by the browser. However, it is important to 
understand the reasons that Struts currently provides only the base file name.

First off, it is entirely up to the browser as to what is provided as the 
original file name. Most browsers do the intelligent thing (see below), and 
provide only the base file name. In these cases, it is simply not possible for 
Struts to provide the full path name, since it is not available as part of the 
request.

Secondly, it is a real security risk for servers to have access to full paths 
within a client file system. This is why most browsers don't supply it. 
Amazingly, though, there are browsers that do provide the full path.

Given the above, if you still feel it's important to have access to the 
original file name, as provided by the browser (maybe, or maybe not, including 
the path), please submit an enhancement request against Struts, and either 
reference this Bugzilla issue or more fully explain what you need (preferably 
the latter!).


I'm afraid I don't understand either the purpose of the suggested setInputStream
() method, or the class to which it pertains. Please submit a separate Bugzilla 
report against either Commons FileUpload or Struts, as appropriate, and explain 
more fully what you are trying to do, and why the current code base does not 
satisfy that need.

I'm marking this report as INVALID (there aren't any better options ;), with 
the expectation that replacements will show up in appropriate places if 
necessary.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to