Multipart files are bound via their parameter name, just like normal * HTTP parameters: i.e. "uploadedFile" to an "uploadedFile" bean property, * invoking a "setUploadedFile" setter method. - *
The type of the target property for a multipart file can be MultipartFile, + *
The type of the target property for a multipart file can be Part, MultipartFile, * byte[], or String. The latter two receive the contents of the uploaded file; * all metadata like original file name, content type, etc are lost in those cases. * @param request request with parameters to bind (can be multipart) * @see org.springframework.web.multipart.MultipartRequest * @see org.springframework.web.multipart.MultipartFile - * @see #bindMultipartFiles + * @see javax.servlet.http.Part * @see #bind(org.springframework.beans.PropertyValues) */ public void bind(WebRequest request) { MutablePropertyValues mpvs = new MutablePropertyValues(request.getParameterMap()); - if (request instanceof NativeWebRequest) { + + if(isMultiPartNativeRequest(request)) { MultipartRequest multipartRequest = ((NativeWebRequest) request).getNativeRequest(MultipartRequest.class); if (multipartRequest != null) { bindMultipart(multipartRequest.getMultiFileMap(), mpvs); + } else if (ClassUtils.hasMethod(HttpServletRequest.class, "getParts")) { + HttpServletRequest multipartSerlvetRequest = ((NativeWebRequest) request) + .getNativeRequest(HttpServletRequest.class); + Servlet3MultiPartBinder binder = new Servlet3MultiPartBinder(); + binder.bindParts(multipartSerlvetRequest, mpvs); } } doBind(mpvs); @@ -121,4 +139,38 @@ public void closeNoCatch() throws BindException { } } + /** + * Check if the request given as parameter is a NativeWebRequest + * and a multipart request (by checking its Content-Type header). + *
If so, this request will be parsed to bind its multiple parts.
+ * @param request request with parameters to bind
+ * @see org.springframework.web.context.request.NativeWebRequest
+ */
+ protected boolean isMultiPartNativeRequest(WebRequest request) {
+
+ return (StringUtils.startsWithIgnoreCase(
+ request.getHeader(CONTENT_TYPE),MULTIPART_CONTENT_TYPE)
+ && request instanceof NativeWebRequest);
+ }
+
+ /**
+ * Encapsulate Part binding code for Servlet 3.0+ only containers.
+ * @see javax.servlet.http.Part
+ */
+ private class Servlet3MultiPartBinder {
+
+ public void bindParts(HttpServletRequest request, MutablePropertyValues mpvs) {
+ try {
+ for(Part part : request.getParts()) {
+ mpvs.add(part.getName(),part);
+ }
+ } catch (IOException ex) {
+ throw new MultipartException("Could not parse multipart servlet request", ex);
+ } catch(ServletException ex) {
+ throw new MultipartException("Could not parse multipart servlet request", ex);
+ }
+ }
+
+ }
+
}
diff --git a/spring-web/src/test/java/org/springframework/web/bind/support/WebRequestDataBinderIntegrationTests.java b/spring-web/src/test/java/org/springframework/web/bind/support/WebRequestDataBinderIntegrationTests.java
new file mode 100644
index 000000000000..355c191ceffa
--- /dev/null
+++ b/spring-web/src/test/java/org/springframework/web/bind/support/WebRequestDataBinderIntegrationTests.java
@@ -0,0 +1,196 @@
+/*
+ * Copyright 2002-2013 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.web.bind.support;
+
+import org.eclipse.jetty.server.Server;
+import org.eclipse.jetty.servlet.ServletContextHandler;
+import org.eclipse.jetty.servlet.ServletHolder;
+import org.junit.AfterClass;
+import org.junit.Before;
+import org.junit.BeforeClass;
+import org.junit.Test;
+
+import org.springframework.core.io.ClassPathResource;
+import org.springframework.core.io.Resource;
+import org.springframework.http.MediaType;
+import org.springframework.http.client.HttpComponentsClientHttpRequestFactory;
+import org.springframework.mock.web.test.MockMultipartFile;
+import org.springframework.util.LinkedMultiValueMap;
+import org.springframework.util.MultiValueMap;
+import org.springframework.util.SocketUtils;
+import org.springframework.web.client.RestTemplate;
+import org.springframework.web.context.request.ServletWebRequest;
+
+import javax.servlet.ServletException;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import javax.servlet.http.Part;
+import java.io.IOException;
+import java.util.List;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+
+/**
+ * @author Brian Clozel
+ */
+public class WebRequestDataBinderIntegrationTests {
+
+ protected static String baseUrl;
+
+ protected static MediaType contentType;
+
+ private static Server jettyServer;
+
+ private RestTemplate template;
+
+ private static PartsServlet partsServlet;
+
+ private static PartListServlet partListServlet;
+
+ @Before
+ public void createTemplate() {
+ template = new RestTemplate(new HttpComponentsClientHttpRequestFactory());
+ }
+
+ @BeforeClass
+ public static void startJettyServer() throws Exception {
+ int port = SocketUtils.findAvailableTcpPort();
+ jettyServer = new Server(port);
+ baseUrl = "http://localhost:" + port;
+ ServletContextHandler handler = new ServletContextHandler();
+
+ partsServlet = new PartsServlet();
+ partListServlet = new PartListServlet();
+
+ handler.addServlet(new ServletHolder(partsServlet), "/parts");
+ handler.addServlet(new ServletHolder(partListServlet), "/partlist");
+ jettyServer.setHandler(handler);
+ jettyServer.start();
+ }
+
+ @AfterClass
+ public static void stopJettyServer() throws Exception {
+ if (jettyServer != null) {
+ jettyServer.stop();
+ }
+ }
+
+ @SuppressWarnings("serial")
+ private abstract static class AbstractStandardMultipartServlet