StandardServletMultipartResolver accepts any HTTP method for multipart

Issue: SPR-16975
master
Juergen Hoeller 6 years ago
parent bf5fe46fa9
commit bac68c8d3f
  1. 20
      spring-web/src/main/java/org/springframework/web/multipart/support/StandardServletMultipartResolver.java

@ -43,13 +43,12 @@ import org.springframework.web.multipart.MultipartResolver;
* *
* <pre class="code"> * <pre class="code">
* public class AppInitializer extends AbstractAnnotationConfigDispatcherServletInitializer { * public class AppInitializer extends AbstractAnnotationConfigDispatcherServletInitializer {
* // ... * // ...
* &#064;Override * &#064;Override
* protected void customizeRegistration(ServletRegistration.Dynamic registration) { * protected void customizeRegistration(ServletRegistration.Dynamic registration) {
* * // Optionally also set maxFileSize, maxRequestSize, fileSizeThreshold
* // Optionally also set maxFileSize, maxRequestSize, fileSizeThreshold * registration.setMultipartConfig(new MultipartConfigElement("/tmp"));
* registration.setMultipartConfig(new MultipartConfigElement("/tmp")); * }
* }
* } * }
* </pre> * </pre>
* *
@ -80,12 +79,7 @@ public class StandardServletMultipartResolver implements MultipartResolver {
@Override @Override
public boolean isMultipart(HttpServletRequest request) { public boolean isMultipart(HttpServletRequest request) {
// Same check as in Commons FileUpload... return StringUtils.startsWithIgnoreCase(request.getContentType(), "multipart/");
if (!"post".equalsIgnoreCase(request.getMethod())) {
return false;
}
String contentType = request.getContentType();
return StringUtils.startsWithIgnoreCase(contentType, "multipart/");
} }
@Override @Override

Loading…
Cancel
Save