Skip to content

WW-4428 Add java.time support to JSON plugin (6.8.x backport) #1616

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: release/struts-6-8-x
Choose a base branch
from
+573 −55

WW-4428 feat(json): add java.time serialization and deserialization s…

ef92392
Select commit
Loading
Failed to load commit list.
Open

WW-4428 Add java.time support to JSON plugin (6.8.x backport) #1616

WW-4428 feat(json): add java.time serialization and deserialization s…
ef92392
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 11, 2026 in 3m 35s

Quality Gate failed

Failed conditions
24 Security Hotspots
42.5% Coverage on New Code (required ≥ 80%)
3.4% Duplication on New Code (required ≤ 3%)
E Security Rating on New Code (required ≥ A)
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 warning on line 132 in core/src/test/java/com/opensymphony/xwork2/validator/URLValidatorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "ActionContext"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGWsf3FRD2v0j-cW1&open=AZzSGWsf3FRD2v0j-cW1&pullRequest=1616

Check warning on line 225 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/parser/ELParser.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code to remove this label and the need for it. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYcj3FRD2v0j-dOt&open=AZzSGYcj3FRD2v0j-dOt&pullRequest=1616

Check warning on line 146 in plugins/portlet/src/main/java/org/apache/struts2/portlet/servlet/PortletServletResponseJSR286.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "@Override" annotation above this method signature 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGZKD3FRD2v0j-dbk&open=AZzSGZKD3FRD2v0j-dbk&pullRequest=1616

Check warning on line 193 in core/src/main/java/org/apache/struts2/views/jsp/iterator/IteratorGeneratorTag.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "isEmpty()" to check whether a "String" is empty or not. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGVc-3FRD2v0j-cHM&open=AZzSGVc-3FRD2v0j-cHM&pullRequest=1616

Check warning on line 93 in core/src/test/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "ActionContext"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGWf-3FRD2v0j-cSJ&open=AZzSGWf-3FRD2v0j-cSJ&pullRequest=1616

Check warning on line 49 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/ServletWriter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGX0H3FRD2v0j-c0z&open=AZzSGX0H3FRD2v0j-c0z&pullRequest=1616

Check warning on line 930 in core/src/main/java/com/opensymphony/xwork2/config/providers/XmlDocConfigurationProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "buildExceptionMappings"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGTMh3FRD2v0j-byx&open=AZzSGTMh3FRD2v0j-byx&pullRequest=1616

Check warning on line 118 in plugins/sitemesh/src/main/java/org/apache/struts2/sitemesh/FreemarkerDecoratorServlet.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "@Override" annotation above this method signature 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGaDT3FRD2v0j-dnY&open=AZzSGaDT3FRD2v0j-dnY&pullRequest=1616

Check warning on line 231 in plugins/oval/src/main/java/org/apache/struts2/oval/interceptor/OValValidationInterceptor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "getCauses"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGZnW3FRD2v0j-dhm&open=AZzSGZnW3FRD2v0j-dhm&pullRequest=1616

Check warning on line 160 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/lang/ELArithmetic.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this "Double" constructor 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYki3FRD2v0j-dUa&open=AZzSGYki3FRD2v0j-dUa&pullRequest=1616

Check failure on line 451 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/JspCompilationContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal ".java" 3 times. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGX703FRD2v0j-c6d&open=AZzSGX703FRD2v0j-c6d&pullRequest=1616

Check warning on line 1683 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/Node.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the synchronized class "Vector" by an unsynchronized one such as "ArrayList" or "LinkedList". 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGXvF3FRD2v0j-cxE&open=AZzSGXvF3FRD2v0j-cxE&pullRequest=1616

Check warning on line 69 in core/src/main/java/com/opensymphony/xwork2/inject/Key.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Provide the parametrized type for this generic. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGS1m3FRD2v0j-brx&open=AZzSGS1m3FRD2v0j-brx&pullRequest=1616

Check warning on line 137 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/parser/TokenMgrError.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this method to simply inherit it. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYT13FRD2v0j-dJO&open=AZzSGYT13FRD2v0j-dJO&pullRequest=1616

Check warning on line 542 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/JspDocumentParser.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code in order to not assign to this loop counter from within the loop body. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGXyK3FRD2v0j-czP&open=AZzSGXyK3FRD2v0j-czP&pullRequest=1616

Check warning on line 307 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/PageDataImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "@Override" annotation above this method signature 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGXqx3FRD2v0j-csB&open=AZzSGXqx3FRD2v0j-csB&pullRequest=1616

Check failure on line 139 in core/src/main/java/com/opensymphony/xwork2/interceptor/ChainingInterceptor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make non-static "includes" private or transient. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGTia3FRD2v0j-b23&open=AZzSGTia3FRD2v0j-b23&pullRequest=1616

Check warning on line 49 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/parser/ELParser.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this deprecated "ALIGN" attribute. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYRq3FRD2v0j-dFi&open=AZzSGYRq3FRD2v0j-dFi&pullRequest=1616

Check warning on line 780 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/parser/ELParser.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Catch Exception instead of Throwable. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYcj3FRD2v0j-dPS&open=AZzSGYcj3FRD2v0j-dPS&pullRequest=1616

Check warning on line 89 in core/src/test/java/com/opensymphony/xwork2/ActionContextTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "ActionContext"; it is deprecated. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGXC93FRD2v0j-chC&open=AZzSGXC93FRD2v0j-chC&pullRequest=1616

Check warning on line 263 in plugins/gxp/src/main/java/org/apache/struts2/views/gxp/AbstractGxp.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace generic exceptions with specific library exceptions or a custom exception. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGZ5S3FRD2v0j-dk3&open=AZzSGZ5S3FRD2v0j-dk3&pullRequest=1616

Check warning on line 720 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/runtime/JspRuntimeLibrary.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this "Long" constructor 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYFB3FRD2v0j-dBs&open=AZzSGYFB3FRD2v0j-dBs&pullRequest=1616

Check warning on line 244 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/lang/ELArithmetic.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this call to a deprecated method, it has been marked for removal. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYki3FRD2v0j-dU2&open=AZzSGYki3FRD2v0j-dU2&pullRequest=1616

Check warning on line 77 in plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/ELFunctionMapper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "@Override" annotation above this method signature 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGX233FRD2v0j-c3t&open=AZzSGX233FRD2v0j-c3t&pullRequest=1616

Check warning on line 219 in plugins/embeddedjsp/src/main/java/org/apache/struts2/el/parser/ELParser.html

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this deprecated "VALIGN" attribute. 

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZzSGYRq3FRD2v0j-dJF&open=AZzSGYRq3FRD2v0j-dJF&pullRequest=1616
View more details on SonarQubeCloud