You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Scott Stanlick (JIRA)" <ji...@apache.org> on 2011/03/27 18:47:06 UTC

[jira] [Commented] (WW-3025) Parameters get lost when file upload over max size allowed

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

Scott Stanlick commented on WW-3025:
------------------------------------

Any movement on this one?  I worked about six hours yesterday on this one before determining the problem was here.  I think the file upload docs should point out this nasty, nasty side effect.  As far as refactoring goes, a more immediate resolution might be to modify the processUpload in JakartaMultiPartRequest to deal with the FileUploadException and populate the processNormalFormField(...) unconditionally.

Also, the workaround link above is dead -- as am I after chasing this nasty pig. :)

Peace,
Scott

> Parameters get lost when file upload over max size allowed
> ----------------------------------------------------------
>
>                 Key: WW-3025
>                 URL: https://issues.apache.org/jira/browse/WW-3025
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Core Interceptors
>    Affects Versions: 2.1.6
>         Environment: All
>            Reporter: Tom Nguyen
>             Fix For: 2.2.x
>
>
> When the uploaded file gets rejected because it's content, size, or because of a general problem an Exception is thrown by the MultiPartRequest class. Exceptions are: InvalidContentTypeException, UnknownSizeException, SizeLimitExceededException, and FileUploadException. This can lead to serious problems within the application because the other parameters from the upload form get lost. Happening in a profile page for example means that the user data is lost this can lead to a security Exception. In other case this usually just involves a OGNL-Exception. Meaning your field data like personal file name is lost. Workaround found in http://henning.kropponline.de/index.php/2009/01/18/struts2-fileuploadbase-exception/, but the the still keep uploading to server, not secured.

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