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

http://issues.apache.org/bugzilla/show_bug.cgi?id=39316





------- Additional Comments From [EMAIL PROTECTED]  2006-04-14 21:03 -------
Well, I understand what you are saying.  But why not let Commons FileUpload pass
this item to struts and let struts decide it's a oversized or not?

So I just write a new CommonsMultipartRequestHandler (since this
CommonsMultipartRequestHandler is not in a nutshell) in our own package and do 
this:

// set maxsize to no limit
upload.setSizeMax(-1);

...

// check the file size, if the file size is greater than the size
// in moduleconfig, set the flag and forward other parameters
        while (iter.hasNext()) {
            FileItem item = (FileItem) iter.next();

            if (item.isFormField()) {
                addTextParameter(request, item);
            } else {
                long size = item.getSize();
                if (size > getSizeMax(ac)) {
                   
request.setAttribute(MultipartRequestHandler.ATTRIBUTE_MAX_LENGTH_EXCEEDED,
Boolean.TRUE);
                    continue;
                }
                
                addFileParameter(item);
            }
        }

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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

Reply via email to