Hi Blake,

the idea is to change the upload-config during runtime.
Not sure how moving it to trinidad-config helps to change it
via an Java API.

-Matthias

On Fri, May 1, 2009 at 7:18 PM, Blake Sullivan
<blake.sulli...@oracle.com> wrote:
> Matthias,
>
> Would things be improved if we moved the configuration to
> trinidad-config.xml and allowed the configuration to be retrieved on a
> per-request basis?
>
> -- Blake Sullivan
>
> Matthias Weßendorf (JIRA) said the following On 5/1/2009 4:25 AM PT:
>
>     [
> https://issues.apache.org/jira/browse/TRINIDAD-1464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12704962#action_12704962
> ]
>
> Matthias Weßendorf commented on TRINIDAD-1464:
> ----------------------------------------------
>
> One possible approach:
> *create a new abstract class in the Trinidad API "FileUploadConfig"
> and define abstract get/set methods for "maxMemory", "maxDiskSpace"
> and "tempDir".
>
> *every implementation of the UploadedFileProcessor that wants to change
> the configuration during runtime should not only implement the actual
> UploadedFileProcessor interface, it also has to implement the abstract
> "FileUploadConfig" class. The UploadFileProcessorImpl (default) would
> do that too.
>
> *since the FileUploadConfig implementation is shared over all sessions
> (application scoped), the implementor (e.g. the UploadFileProcessorImpl)
> needs to be aware of writing thread-safety code, when *setting* the new
> value...
>
> possible usage in an action-method:
> ...
> FileUploadConfig fuc = null;
> UploadedFileProcessor ufp = requestCtx.getUploadedFileProcessor();
> if(ufp instanceof FileUploadConfig)
> {
>  fuc = (FileUploadConfig) ufp;
>  fuc.setMax.....(....);
> }
> ...
>
>
>
> max upload size cannot be reconfigured after app is deployed
> ------------------------------------------------------------
>
>                 Key: TRINIDAD-1464
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-1464
>             Project: MyFaces Trinidad
>          Issue Type: New Feature
>    Affects Versions:  1.2.11-core
>            Reporter: Matthias Weßendorf
>            Assignee: Matthias Weßendorf
>
> Currently the only way to configure the max upload size it to set context
> parameters in web.xml. Some container do not provide any support for
> changing
> web.xml during or after deployment...
> Applications needs to be able to change the max upload size after
> deployment.
> This could be done by providing an implementation of
> org.apache.myfaces.trinidad.webapp.UploadedFileProcessor. However that
> would be a relatively simple variation on
> org.apache.myfaces.trinidadinternal.config.upload.UploadedFileProcessorImpl
> We need to find a good way to dynamically change those configuration
> settings
>
>
>
>



-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

Reply via email to