diff --git a/test/test-cases/regression/action-disruptive.json b/test/test-cases/regression/action-disruptive.json index 9ac4e711..3c40cb35 100644 --- a/test/test-cases/regression/action-disruptive.json +++ b/test/test-cases/regression/action-disruptive.json @@ -26,7 +26,7 @@ "SecRuleEngine On", "SecDebugLog \/tmp\/modsec_debug.log", "SecDefaultAction \"phase:2,deny,status:404\"", - "SecAction \"'id:'1',phase:request,nolog,t:none\"" + "SecAction \"id:'1',phase:request,nolog,t:none\"" ] }, { diff --git a/test/test-cases/regression/action-id.json b/test/test-cases/regression/action-id.json index c10d226e..ff311310 100644 --- a/test/test-cases/regression/action-id.json +++ b/test/test-cases/regression/action-id.json @@ -266,7 +266,7 @@ ] }, "expected":{ - "debug_log": "trim: \"value2\"" + "parser_error": "Rules error. File: action-id.json. Line: 4. Column: 35. invalid character '" }, "rules":[ "SecRuleEngine On", diff --git a/test/test-cases/regression/actions.json b/test/test-cases/regression/actions.json index 1fbc32f4..fefd4dc4 100644 --- a/test/test-cases/regression/actions.json +++ b/test/test-cases/regression/actions.json @@ -64,7 +64,7 @@ "enabled": 1, "version_min": 300000, "version_max": 0, - "title": "actions :: trim,redirect:http://www.google.com", + "title": "actions :: trim,redirect:'http://www.google.com'", "client": { "ip": "200.249.12.31", "port": 2313 @@ -119,7 +119,7 @@ "SecRuleEngine On", "SecDebugLog \/tmp\/modsec_debug.log", "SecDebugLogLevel 9", - "SecRule ARGS \"@contains test\" \"id:1,t:trim,redirect:http://www.google.com\"" + "SecRule ARGS \"@contains test\" \"id:1,t:trim,redirect:'http://www.google.com'\"" ] }, { @@ -181,7 +181,7 @@ "SecRuleEngine On", "SecDebugLog \/tmp\/modsec_debug.log", "SecDebugLogLevel 9", - "SecRule ARGS \"@contains test\" \"id:1,t:trim,status:500,redirect:http://www.google.com\"" + "SecRule ARGS \"@contains test\" \"id:1,t:trim,status:500,redirect:'http://www.google.com'\"" ] }, { diff --git a/test/test-cases/regression/config-secdefaultaction.json b/test/test-cases/regression/config-secdefaultaction.json index b35bfeb3..6f83e680 100644 --- a/test/test-cases/regression/config-secdefaultaction.json +++ b/test/test-cases/regression/config-secdefaultaction.json @@ -285,14 +285,14 @@ "expected":{ "audit_log":"", "debug_log":"Request was relevant to be saved.", - "http_code": 500 + "http_code": 302 }, "rules":[ "SecRuleEngine On", "SecDebugLog \/tmp\/modsec_debug.log", "SecDebugLogLevel 9", - "SecDefaultAction \"phase:2,log,auditlog,status:500\"", - "SecRule REQUEST_HEADERS \"@contains PHPSESSID\" \"phase:2,id:1,redirect:http://www.google.com\"", + "SecDefaultAction \"phase:2,log,auditlog,status:302\"", + "SecRule REQUEST_HEADERS \"@contains PHPSESSID\" \"phase:2,id:1,redirect:'http://www.google.com'\"", "SecRule TX \"@contains to_test\" \"id:2,t:lowercase,t:none\"" ] } diff --git a/test/test-cases/regression/secruleengine.json b/test/test-cases/regression/secruleengine.json index 75526c5e..715c1592 100644 --- a/test/test-cases/regression/secruleengine.json +++ b/test/test-cases/regression/secruleengine.json @@ -28,7 +28,7 @@ "SecDebugLog \/tmp\/modsec_debug.log", "SecRuleEngine Off", "SecDefaultAction \"phase:2,deny,status:404\"", - "SecAction \"'id:'1',phase:request,nolog,t:none\"" + "SecAction \"id:'1',phase:request,nolog,t:none\"" ] }, { @@ -60,7 +60,7 @@ "SecDebugLog \/tmp\/modsec_debug.log", "SecRuleEngine Off", "SecDefaultAction \"phase:2,deny,status:404\"", - "SecAction \"'id:'1',phase:request,nolog,t:none\"" + "SecAction \"id:'1',phase:request,nolog,t:none\"" ] }, {