Skip to content

Commit f72106d

Browse files
committed
Make test clearer in regression/request-body-parser-multipart-crlf.json
This test expects MULTIPART_STRICT_ERROR because of inconsistency of CRLF or LF after boundaries.
1 parent c795b09 commit f72106d

1 file changed

Lines changed: 4 additions & 7 deletions

File tree

test/test-cases/regression/request-body-parser-multipart-crlf.json

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
{
33
"enabled": 1,
44
"version_min": 300000,
5-
"title": "multipart parser (final CRLF)",
5+
"title": "multipart parser (final boundary CRLF, other boundary LF)",
66
"client": {
77
"ip": "200.249.12.31",
88
"port": 123
@@ -16,7 +16,7 @@
1616
"Host": "localhost",
1717
"User-Agent": "curl/7.38.0",
1818
"Accept": "*/*",
19-
"Content-Length": "303",
19+
"Content-Length": "304",
2020
"Content-Type": "multipart/form-data; boundary=---------------------------69343412719991675451336310646",
2121
"Expect": "100-continue"
2222
},
@@ -37,7 +37,7 @@
3737
"2.1\r\n",
3838
"2.2\r\n",
3939
"2.3\r\n",
40-
"-----------------------------69343412719991675451336310646--\n"
40+
"-----------------------------69343412719991675451336310646--\r\n"
4141
]
4242
},
4343
"response": {
@@ -58,10 +58,7 @@
5858
"rules": [
5959
"SecRuleEngine On",
6060
"SecRequestBodyAccess On",
61-
"SecRule MULTIPART_STRICT_ERROR \"@eq 1\" \"phase:2,deny,id:500055\"",
62-
"SecRule MULTIPART_UNMATCHED_BOUNDARY \"@eq 1\" \"phase:2,deny,id:500056\"",
63-
"SecRule REQBODY_PROCESSOR_ERROR \"@eq 1\" \"phase:2,deny,id:500057\"",
64-
"SecRule ARGS_POST \"@eq 1231\" \"phase:2,deny,id:500067\""
61+
"SecRule MULTIPART_STRICT_ERROR \"@eq 1\" \"phase:2,deny,id:500055\""
6562
]
6663
}
6764
]

0 commit comments

Comments
 (0)