From e1cff89b484d5a234384496e9f1f63030cd350bb Mon Sep 17 00:00:00 2001 From: Felipe Zimmerle Date: Fri, 17 Jul 2015 15:38:04 -0300 Subject: [PATCH] Adds support to FILES_NAMES variable --- src/request_body_processor/multipart.cc | 1 + .../regression/variable-FILES_NAMES.json | 63 +++++++++++++++++++ 2 files changed, 64 insertions(+) create mode 100644 test/test-cases/regression/variable-FILES_NAMES.json diff --git a/src/request_body_processor/multipart.cc b/src/request_body_processor/multipart.cc index 1fed765f..3562195d 100644 --- a/src/request_body_processor/multipart.cc +++ b/src/request_body_processor/multipart.cc @@ -210,6 +210,7 @@ bool Multipart::process(std::string data) { if (m.filename.empty() == false) { variables.emplace("FILES:" + m.name, m.filename); + variables.emplace("FILES_NAMES:" + m.name, m.name); } } diff --git a/test/test-cases/regression/variable-FILES_NAMES.json b/test/test-cases/regression/variable-FILES_NAMES.json new file mode 100644 index 00000000..d2052e59 --- /dev/null +++ b/test/test-cases/regression/variable-FILES_NAMES.json @@ -0,0 +1,63 @@ +[ + { + "enabled":1, + "version_min":300000, + "title":"Testing Variables :: FILES_NAMES (1/1)", + "client":{ + "ip":"200.249.12.31", + "port":123 + }, + "server":{ + "ip":"200.249.12.31", + "port":80 + }, + "request":{ + "headers":{ + "Host":"localhost", + "User-Agent":"curl/7.38.0", + "Accept":"*/*", + "Content-Length":"330", + "Content-Type":"multipart/form-data; boundary=--------------------------756b6d74fa1a8ee2", + "Expect":"100-continue" + }, + "uri":"/", + "protocol":"POST", + "body":[ + "--------------------------756b6d74fa1a8ee2", + "Content-Disposition: form-data; name=\"name\"", + "", + "test", + "--------------------------756b6d74fa1a8ee2", + "Content-Disposition: form-data; name=\"filedata\"; filename=\"small_text_file.txt\"", + "Content-Type: text/plain", + "", + "This is a very small test file..", + "--------------------------756b6d74fa1a8ee2", + "Content-Disposition: form-data; name=\"filedata\"; filename=\"small_text_file.txt\"", + "Content-Type: text/plain", + "", + "This is another very small test file..", + "--------------------------756b6d74fa1a8ee2--" + ] + }, + "response":{ + "headers":{ + "Date":"Mon, 13 Jul 2015 20:02:41 GMT", + "Last-Modified":"Sun, 26 Oct 2014 22:33:37 GMT", + "Content-Type":"text/html" + }, + "body":[ + "no need." + ] + }, + "expected":{ + "debug_log":"T \\(1\\) trim: \"filedata" + }, + "rules":[ + "SecRuleEngine On", + "SecDebugLog \/tmp\/modsec_debug.log", + "SecDebugLogLevel 9", + "SecRule FILES_NAMES \"@contains small_text_file.txt\" \"phase:3,pass,t:trim\"" + ] + } +] \ No newline at end of file