Skip to content

WW-5659 Resolve lazy interceptor params per invocation - #1816

Open
lukaszlenart wants to merge 23 commits into
mainfrom
WW-5659-lazy-params-request-scoping
Open

WW-5659 Resolve lazy interceptor params per invocation#1816
lukaszlenart wants to merge 23 commits into
mainfrom
WW-5659-lazy-params-request-scoping

WW-5659 fix(core): stop seeding the interceptor with raw lazy express…

83d2b61
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 27, 2026 in 1m 20s

Annotations

Check warning on line 59 in core/src/main/java/org/apache/struts2/DefaultActionInvocation.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this “Monster Class” into smaller and more specialized ones to reduce its dependencies on other classes from 22 to the maximum authorized 20 or less.

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZ-jOAxGIe-g1Dd6oyJA&open=AZ-jOAxGIe-g1Dd6oyJA&pullRequest=1816

Check warning on line 651 in core/src/test/java/org/apache/struts2/interceptor/ActionFileUploadInterceptorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This block of commented-out lines of code should be removed.

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZ-jOAuqIe-g1Dd6oyI1&open=AZ-jOAuqIe-g1Dd6oyI1&pullRequest=1816

Check failure on line 126 in core/src/main/java/org/apache/struts2/interceptor/AbstractFileUploadInterceptor.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=AZ-jOAwMIe-g1Dd6oyI-&open=AZ-jOAwMIe-g1Dd6oyI-&pullRequest=1816

Check failure on line 233 in core/src/main/java/org/apache/struts2/interceptor/ActionFileUploadInterceptor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZ-jOAwjIe-g1Dd6oyI_&open=AZ-jOAwjIe-g1Dd6oyI_&pullRequest=1816

Check failure on line 67 in core/src/test/java/org/apache/struts2/interceptor/LazyParamsAllowlistTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a "super.setUp()" call to this method.

See more on https://sonarcloud.io/project/issues?id=apache_struts&issues=AZ-jOAvgIe-g1Dd6oyI7&open=AZ-jOAvgIe-g1Dd6oyI7&pullRequest=1816

Check warning on line 68 in core/src/main/java/org/apache/struts2/interceptor/WithLazyParams.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=AZ-jOAv7Ie-g1Dd6oyI8&open=AZ-jOAv7Ie-g1Dd6oyI8&pullRequest=1816