Merge pull request #3306 from airween/v3/time_mon_fix

fix: align TIME_MON variable's behavior
owasp-modsecurity · Nov 24, 2024 · d9101a4 · d9101a4
2 parents fd45641 + daf550e
commit d9101a4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/variables/time_mon.cc b/src/variables/time_mon.cc
@@ -46,7 +46,7 @@ void TimeMon::evaluate(Transaction *transaction,
     struct tm timeinfo;
     localtime_r(&timer, &timeinfo);
 
-    transaction->m_variableTimeMin.assign(std::to_string(timeinfo.tm_mon));
+    transaction->m_variableTimeMin.assign(std::to_string(timeinfo.tm_mon + 1));
 
     l->push_back(new VariableValue(&m_retName,
         &transaction->m_variableTimeMin));

diff --git a/test/test-cases/regression/variable-TIME_MON.json b/test/test-cases/regression/variable-TIME_MON.json
@@ -33,7 +33,8 @@
       ]
     },
     "expected":{
-      "debug_log":"Target value: \"([0-9]+)\" \\(Variable: TIME_MON\\)"
+      "http_code": 200,
+      "debug_log":"Target value: \"([1-9]|1[012])\" \\(Variable: TIME_MON\\)"
     },
     "rules":[
       "SecRuleEngine On",