Skip to content

Enable pre profile update action to allow modifying user claims at profile update#8115

Closed
Lashen1227 wants to merge 14 commits into
wso2:masterfrom
Lashen1227:feat/pre-profile-update-claims
Closed

Enable pre profile update action to allow modifying user claims at profile update#8115
Lashen1227 wants to merge 14 commits into
wso2:masterfrom
Lashen1227:feat/pre-profile-update-claims

feat: enhance claim validation in PreUpdateProfileResponseProcessor t…

04d3fd3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 26, 2026 in 2m 7s

Annotations

Check warning on line 445 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these unused method parameters "initiatorType", "simpleMultiValuedClaimsToBeRemoved".

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqz&open=AZ5ioOs3xRHZv9iNmJqz&pullRequest=8115

Check warning on line 635 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string'

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq3&open=AZ5ioOs3xRHZv9iNmJq3&pullRequest=8115

Check warning on line 385 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqv&open=AZ5ioOs3xRHZv9iNmJqv&pullRequest=8115

Check warning on line 420 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqx&open=AZ5ioOs3xRHZv9iNmJqx&pullRequest=8115

Check warning on line 325 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Method has 9 parameters, which is greater than 7 authorized.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqq&open=AZ5ioOs3xRHZv9iNmJqq&pullRequest=8115

Check failure on line 128 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.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=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqr&open=AZ5ioOs3xRHZv9iNmJqr&pullRequest=8115

Check failure on line 325 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqu&open=AZ5ioOs3xRHZv9iNmJqu&pullRequest=8115

Check failure on line 590 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq1&open=AZ5ioOs3xRHZv9iNmJq1&pullRequest=8115

Check warning on line 95 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused method parameter "event".

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqn&open=AZ5ioOpHxRHZv9iNmJqn&pullRequest=8115

Check warning on line 692 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq4&open=AZ5ioOs3xRHZv9iNmJq4&pullRequest=8115

Check warning on line 77 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor your code to get this URI from a customizable parameter.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq5&open=AZ5ioOs3xRHZv9iNmJq5&pullRequest=8115

Check warning on line 547 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq0&open=AZ5ioOs3xRHZv9iNmJq0&pullRequest=8115

Check warning on line 449 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'org.wso2.carbon.identity.action.execution.api.exception.ActionExecutionResponseProcessorException', as it cannot be thrown from method's body.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqy&open=AZ5ioOs3xRHZv9iNmJqy&pullRequest=8115

Check warning on line 246 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string'

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqt&open=AZ5ioOs3xRHZv9iNmJqt&pullRequest=8115

Check failure on line 251 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.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=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqo&open=AZ5ioOpHxRHZv9iNmJqo&pullRequest=8115

Check warning on line 70 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor your code to get this URI from a customizable parameter.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqp&open=AZ5ioOpHxRHZv9iNmJqp&pullRequest=8115

Check warning on line 95 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileRequestBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'org.wso2.carbon.identity.action.execution.api.exception.ActionExecutionRequestBuilderException', as it cannot be thrown from method's body.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOpHxRHZv9iNmJqm&open=AZ5ioOpHxRHZv9iNmJqm&pullRequest=8115

Check warning on line 629 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string'

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJq2&open=AZ5ioOs3xRHZv9iNmJq2&pullRequest=8115

Check warning on line 168 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof String string'

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqs&open=AZ5ioOs3xRHZv9iNmJqs&pullRequest=8115

Check warning on line 391 in components/user-mgt/org.wso2.carbon.identity.user.pre.update.profile.action/src/main/java/org/wso2/carbon/identity/user/pre/update/profile/action/internal/execution/PreUpdateProfileResponseProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified.

See more on https://sonarcloud.io/project/issues?id=wso2_carbon-identity-framework&issues=AZ5ioOs3xRHZv9iNmJqw&open=AZ5ioOs3xRHZv9iNmJqw&pullRequest=8115