You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Jochen Wiedmann (Jira)" <ji...@apache.org> on 2019/09/23 14:13:00 UTC

[jira] [Updated] (FILEUPLOAD-300) Create request wrapper

     [ https://issues.apache.org/jira/browse/FILEUPLOAD-300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jochen Wiedmann updated FILEUPLOAD-300:
---------------------------------------
    External issue URL: https://github.com/apache/commons-fileupload/pull/22
    Remaining Estimate: 8h
     Original Estimate: 8h

> Create request wrapper
> ----------------------
>
>                 Key: FILEUPLOAD-300
>                 URL: https://issues.apache.org/jira/browse/FILEUPLOAD-300
>             Project: Commons FileUpload
>          Issue Type: Task
>    Affects Versions: 1.3.3
>            Reporter: Jochen Wiedmann
>            Assignee: Jochen Wiedmann
>            Priority: Minor
>             Fix For: 1.4
>
>   Original Estimate: 8h
>  Remaining Estimate: 8h
>
>  
> As of this writing, there is now possibility to have request specific customizations, or data.
> This is mostly, because the user is typically interacting with the FileUploadBase object only, making seemingly simple issues, like the wish for a request speific value of fileSieMax ([Pull Request 22 on Github[https://github.com/apache/commons-fileupload/pull/22]] unnecessarily complex.
>  
> As the current master provides an opportunity for API changes, we will:
>  
> 1.) Split the FileItemIteratorImpl class, as well as its inner class FileItemStreamImpl out off the FileUploadBase class into separate objects.
> 2.) Defer initialization of the FileItemIteratorImpl class as long as possible, making the fields "multi", "notifier", and "boundary" non-final.
> 3.) Extend the FileItemIterator interface, and (in turn), the FileItemIteratorImpl object with methods like getFileSizeMax(), setFileSizeMax(long), etc.
> 4.) Change the API to enforce use of the FileItemIterator (and thus, the user noticing it's existence).
>  
> As a result we will have a greatly reduced FileUploadBase class, which is much better maintainable, than the current version.
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)