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)