Skip to content

Prototype: Evaluate alternatives to YAJL for JSON processing #3539

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Easton97-Jens wants to merge 15 commits into
base: v3/master
Choose a base branch
from

Update test.yml

0131c8a
Select commit
Loading
Failed to load commit list.
Open

Prototype: Evaluate alternatives to YAJL for JSON processing #3539

Update test.yml
0131c8a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 8, 2026 in 3m 3s

Quality Gate failed

Failed conditions
7 Security Hotspots
3.3% Duplication on New Code (required ≤ 3%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 8255 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements. 

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

Check warning on line 4090 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::type" with "std::common_type_t". 

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

Check warning on line 5973 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 15625 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the init-statement to declare "N" inside the if statement. 

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

Check warning on line 16360 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Combine this function with its overloads by using "if constexpr". 

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

Check warning on line 9100 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_same_v". 

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

Check warning on line 4734 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_same_v". 

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

Check warning on line 12932 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_base_of_v". 

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

Check warning on line 14802 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 252 in src/modsecurity.cc

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this loop so that it is less error-prone. 

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

Check warning on line 5223 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_nothrow_move_assignable_v". 

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

Check warning on line 22984 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make this member overloaded operator a hidden friend. 

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

Check warning on line 16219 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Uppercase the literal suffix. 

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

Check warning on line 19173 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this loop so that it is less error-prone. 

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

Check warning on line 13630 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 9025 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check failure on line 24736 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this #undef directive that undefines a non-existing macro. 

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

Check warning on line 23299 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make the type of this parameter a reference-to-const. The current type of "o" is "std::ostream &". 

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

Check failure on line 2829 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this macro by "const", "constexpr" or an "enum". 

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

Check failure on line 11119 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements. 

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

Check warning on line 3869 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_same_v". 

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

Check warning on line 16179 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract the assignment from this expression. 

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

Check warning on line 17915 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 12180 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 3546 in third_party/nlohmann_json/include/nlohmann/json.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the use of "::value" with "std::is_same_v". 

See more on https://sonarcloud.io/project/issues?id=owasp-modsecurity_ModSecurity&issues=AZ1uOanP6u2ZGBQk4nhx&open=AZ1uOanP6u2ZGBQk4nhx&pullRequest=3539
View more details on SonarQubeCloud