Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 364283

Summary:	can't parse the servlet multipart-config for the web.xml.
Product:	[RT] Jetty	Reporter:	d0ngw <agile.java>
Component:	server	Assignee:	Jan Bartel <janb>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	P3	CC:	agile.java, janb, jetty-inbox
Version:	8.0.4
Target Milestone:	7.5.x
Hardware:	All
OS:	All
Whiteboard:

Description d0ngw

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

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

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.