Skip to content

Commit c7a6daf

Browse files
authored
Merge pull request #1169 from apache/fix/WW-5500
WW-5500 Extends pattern to validate multipart uploads
2 parents 370c7ce + 575e4e8 commit c7a6daf

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public class Dispatcher {
112112
*/
113113
public static final String REQUEST_POST_METHOD = "POST";
114114

115-
public static final String MULTIPART_FORM_DATA_REGEX = "^multipart/form-data(?:\\s*;\\s*boundary=[0-9a-zA-Z'()+_,\\-./:=?]{1,70})?(?:\\s*;\\s*charset=[a-zA-Z\\-0-9]{3,14})?";
115+
public static final String MULTIPART_FORM_DATA_REGEX = "^multipart/form-data(?:\\s*;\\s*boundary=[0-9a-zA-Z'\"()+_,\\-./:=?]{1,70})?(?:\\s*;\\s*charset=[a-zA-Z\\-0-9]{3,14})?";
116116

117117
private static final String CONFIG_SPLIT_REGEX = "\\s*,\\s*";
118118

core/src/test/java/org/apache/struts2/dispatcher/DispatcherTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -343,6 +343,9 @@ public void testIsMultipartRequest() {
343343

344344
req.setContentType("Multipart/Form-Data ; boundary=---------------------------207103069210263;charset=UTF-16LE");
345345
assertTrue(dispatcher.isMultipartRequest(req));
346+
347+
req.setContentType("multipart/form-data; boundary=\"----=_Part_38_1092302434.1734807780737\"");
348+
assertTrue(dispatcher.isMultipartRequest(req));
346349
}
347350

348351
@Test

0 commit comments

Comments
 (0)