litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fully consume input stream so that error message can be displayed in file upload

Browse source
This commit is contained in:
Graham Triggs 2017-09-27 15:59:01 +01:00
parent e08a8c7715
commit 2979cdadae
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
api/src/main/java/edu/cornell/mannlib/vitro/webapp/controller/MultipartRequestWrapper.java
View file

@ -4,6 +4,7 @@ package edu.cornell.mannlib.vitro.webapp.controller;
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.io.UnsupportedEncodingException;
import java.net.URLDecoder;
import java.util.ArrayList;
@ -190,6 +191,15 @@ public class MultipartRequestWrapper extends HttpServletRequestWrapper {
try {
return upload.parseRequest(req);
} catch (FileSizeLimitExceededException | SizeLimitExceededException e) {
// Fully consume input stream, so that error message is displayed
try {
InputStream in = req.getInputStream();
byte[] buffer = new byte[4096];
while (in.read(buffer) != -1) ;
} catch (IOException ioe) {
// Ignore any errors at this point
}
if (strategy.stashFileSizeException()) {
req.setAttribute(ATTRIBUTE_FILE_SIZE_EXCEPTION, e);
return Collections.emptyList();