Skip to content

V3/master3017 - #3647

Merged
airween merged 58 commits into
v3/masterfrom
v3/master3017
Sep 29, 2026
Merged

airween merged 58 commits into
v3/masterfrom
v3/master3017

Change release version to v3.0.17

da9fe3d
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 29, 2026 in 3m 11s

Annotations

Check failure on line 847 in src/request_body_processor/multipart.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IPR3f2ysA7ZALn6&open=AaDu2IPR3f2ysA7ZALn6&pullRequest=3647

Check failure on line 47 in src/request_body_processor/xml.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2INm3f2ysA7ZALnw&open=AaDu2INm3f2ysA7ZALnw&pullRequest=3647

Check failure on line 197 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Global variables should be const.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALn_&open=AaDu2ISL3f2ysA7ZALn_&pullRequest=3647

Check warning on line 1124 in src/request_body_processor/multipart.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IPR3f2ysA7ZALn8&open=AaDu2IPR3f2ysA7ZALn8&pullRequest=3647

Check warning on line 9338 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoS&open=AaDu2ISL3f2ysA7ZALoS&pullRequest=3647

Check warning on line 89 in src/request_body_processor/multipart.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this structure so it has no more than 20 fields, rather than the 22 it currently has.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ILG3f2ysA7ZALnu&open=AaDu2ILG3f2ysA7ZALnu&pullRequest=3647

Check warning on line 9276 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoQ&open=AaDu2ISL3f2ysA7ZALoQ&pullRequest=3647

Check warning on line 4463 in src/parser/seclang-parser.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IXe3f2ysA7ZALoc&open=AaDu2IXe3f2ysA7ZALoc&pullRequest=3647

Check warning on line 1943 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this C-style array with "std::vector" (for dynamic size), or "std::array" (for static size)

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoF&open=AaDu2ISL3f2ysA7ZALoF&pullRequest=3647

Check warning on line 847 in src/request_body_processor/multipart.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this invocation of "strstr" with a call to the "find" string method.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IPR3f2ysA7ZALn3&open=AaDu2IPR3f2ysA7ZALn3&pullRequest=3647

Check warning on line 9890 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make sure use of "strlen" is safe here.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoU&open=AaDu2ISL3f2ysA7ZALoU&pullRequest=3647

Check failure on line 45 in tools/gen-html-entities.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Path Traversal via faulty LLM-supplied CLI arguments

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IZp3f2ysA7ZALof&open=AaDu2IZp3f2ysA7ZALof&pullRequest=3647

Check warning on line 9224 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoM&open=AaDu2ISL3f2ysA7ZALoM&pullRequest=3647

Check failure on line 45 in src/operators/rx.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 26 to the 25 allowed.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IEd3f2ysA7ZALnk&open=AaDu2IEd3f2ysA7ZALnk&pullRequest=3647

Check warning on line 445 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this C-style array with "std::vector" (for dynamic size), or "std::array" (for static size)

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoC&open=AaDu2ISL3f2ysA7ZALoC&pullRequest=3647

Check warning on line 65 in tools/gen-html-entities.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "min()" instead of sorting to find this value.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IZp3f2ysA7ZALoe&open=AaDu2IZp3f2ysA7ZALoe&pullRequest=3647

Check warning on line 1447 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this C-style array with "std::vector" (for dynamic size), or "std::array" (for static size)

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoE&open=AaDu2ISL3f2ysA7ZALoE&pullRequest=3647

Check warning on line 358 in src/request_body_processor/multipart.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IPR3f2ysA7ZALn0&open=AaDu2IPR3f2ysA7ZALn0&pullRequest=3647

Check warning on line 105 in src/request_body_processor/multipart.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use the constructor's initializer list for data member "m_filename_language". Use the in-class initializer instead.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ILG3f2ysA7ZALns&open=AaDu2ILG3f2ysA7ZALns&pullRequest=3647

Check warning on line 9253 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoN&open=AaDu2ISL3f2ysA7ZALoN&pullRequest=3647

Check warning on line 64 in src/operators/rx.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IEd3f2ysA7ZALnl&open=AaDu2IEd3f2ysA7ZALnl&pullRequest=3647

Check warning on line 3655 in src/parser/seclang-parser.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IXe3f2ysA7ZALoY&open=AaDu2IXe3f2ysA7ZALoY&pullRequest=3647

Check warning on line 5379 in src/parser/seclang-scanner.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this C-style array with "std::vector" (for dynamic size), or "std::array" (for static size)

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2ISL3f2ysA7ZALoH&open=AaDu2ISL3f2ysA7ZALoH&pullRequest=3647

Check warning on line 373 in src/request_body_processor/multipart.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IPR3f2ysA7ZALn1&open=AaDu2IPR3f2ysA7ZALn1&pullRequest=3647

Check warning on line 64 in src/operators/rx_global.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement.

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AaDu2IAl3f2ysA7ZALnj&open=AaDu2IAl3f2ysA7ZALnj&pullRequest=3647