Skip to content

WW-5618 Add configurable limits to JSON plugin #1625

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
lukaszlenart wants to merge 1 commit into
base: main
Choose a base branch
from
+1,314 −367

WW-5618 feat(json): add configurable limits to JSON plugin

9d6721f
Select commit
Loading
Failed to load commit list.
Open

WW-5618 Add configurable limits to JSON plugin #1625

WW-5618 feat(json): add configurable limits to JSON plugin
9d6721f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 17, 2026 in 1m 31s

Quality Gate passed

Issues
15 New issues
0 Accepted issues

Measures
0 Security Hotspots
84.2% Coverage on New Code
0.8% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 427 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Format specifiers should be used instead of string concatenation. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhL&open=AZz74QbEq1HksJe4IPhL&pullRequest=1625

Check failure on line 454 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhM&open=AZz74QbEq1HksJe4IPhM&pullRequest=1625

Check warning on line 243 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhD&open=AZz74QbEq1HksJe4IPhD&pullRequest=1625

Check warning on line 194 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof Date date' 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhG&open=AZz74QbEq1HksJe4IPhG&pullRequest=1625

Check warning on line 345 in plugins/json/src/main/java/org/apache/struts2/json/JSONUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74Qa6q1HksJe4IPhC&open=AZz74Qa6q1HksJe4IPhC&pullRequest=1625

Check warning on line 196 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof Calendar calendar' 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhH&open=AZz74QbEq1HksJe4IPhH&pullRequest=1625

Check warning on line 413 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Format specifiers should be used instead of string concatenation. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhK&open=AZz74QbEq1HksJe4IPhK&pullRequest=1625

Check warning on line 461 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhE&open=AZz74QbEq1HksJe4IPhE&pullRequest=1625

Check warning on line 180 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof Boolean boolean' 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhF&open=AZz74QbEq1HksJe4IPhF&pullRequest=1625

Check failure on line 102 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QaOq1HksJe4IPg7&open=AZz74QaOq1HksJe4IPg7&pullRequest=1625

Check failure on line 74 in plugins/json/src/main/java/org/apache/struts2/json/JSONInterceptor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make non-static "jsonUtil" transient or serializable. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QaFq1HksJe4IPgw&open=AZz74QaFq1HksJe4IPgw&pullRequest=1625

Check failure on line 408 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhJ&open=AZz74QbEq1HksJe4IPhJ&pullRequest=1625

Check warning on line 329 in plugins/json/src/main/java/org/apache/struts2/json/JSONUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74Qa6q1HksJe4IPhA&open=AZz74Qa6q1HksJe4IPhA&pullRequest=1625

Check failure on line 152 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz8V8bEOgK6FF7Sg3p2&open=AZz8V8bEOgK6FF7Sg3p2&pullRequest=1625

Check failure on line 228 in plugins/json/src/main/java/org/apache/struts2/json/StrutsJSONWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 36 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZz74QbEq1HksJe4IPhI&open=AZz74QbEq1HksJe4IPhI&pullRequest=1625
View more details on SonarQubeCloud