Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364283 - can't parse the servlet multipart-config for the web.xml.
Summary: can't parse the servlet multipart-config for the web.xml.
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: server (show other bugs)
Version: 8.0.4   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 7.5.x   Edit
Assignee: Jan Bartel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-21 05:04 EST by d0ngw CLA
Modified: 2011-11-21 21:44 EST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description d0ngw CLA 2011-11-21 05:04:56 EST 
Build Identifier: jetty-8.0.4.v20111024

The MultipartConfigElement can't be inited with  the the multipart-config defined at the web.xml.
Maybe the xml parser using the wrong Node object on line 555 of StandardDescriptorProcessor.java ?

if (multipart != null)
        {
            String location = node.getString("location", false, true);
            String maxFile = node.getString("max-file-size", false, true);
            String maxRequest = node.getString("max-request-size", false, true);
            String threshold = node.getString("file-size-threshold",false,true); 

and I think it should be:

if (multipart != null)
        {
            String location = multipart.getString("location", false, true);
            String maxFile = multipart.getString("max-file-size", false, true);
            String maxRequest = multipart.getString("max-request-size", false, true);
            String threshold = multipart.getString("file-size-threshold",false,true); 


Reproducible: Always
Comment 1 Jan Bartel CLA 2011-11-21 20:18:42 EST 
Drat. That serves me right for testing with annotations instead! 
Fixed for jetty-8 next release, scheduled as jetty-8.1.0.
Comment 2 d0ngw CLA 2011-11-21 21:44:45 EST 
So magic speed,thanks.

(In reply to comment #1)
> Drat. That serves me right for testing with annotations instead! 
> Fixed for jetty-8 next release, scheduled as jetty-8.1.0.