Skip to content

Potential fix for pull request finding

ddef9b8
Select commit
Loading
Failed to load commit list.
Open

fix(QTDI-2954): fix possible NPE in VaultClient #1230

Potential fix for pull request finding
ddef9b8
Select commit
Loading
Failed to load commit list.
sonar-rnd / SonarQube Code Analysis failed May 28, 2026 in 1m 47s

Quality Gate failed

Failed Quality Gate failed

  • 0.00% Coverage on New Code (is less than 80.00%)
  • 4 New Issues (is greater than 0)

Project ID: org.talend.sdk.component:component-runtime

View in SonarQube

Annotations

Check warning on line 429 in vault-client/src/main/java/org/talend/sdk/components/vault/client/VaultClient.java

See this annotation in the file changed.

@sonar-rnd sonar-rnd / SonarQube Code Analysis

vault-client/src/main/java/org/talend/sdk/components/vault/client/VaultClient.java#L429

Replace this usage of "ErrorPayload.class.isInstance()" with "instanceof ErrorPayload".

Check warning on line 491 in vault-client/src/main/java/org/talend/sdk/components/vault/client/VaultClient.java

See this annotation in the file changed.

@sonar-rnd sonar-rnd / SonarQube Code Analysis

vault-client/src/main/java/org/talend/sdk/components/vault/client/VaultClient.java#L491

Replace this usage of "WebApplicationException.class.isInstance()" with "instanceof WebApplicationException".

Check warning on line 362 in vault-client/src/test/java/org/talend/sdk/components/vault/client/VaultClientTest.java

See this annotation in the file changed.

@sonar-rnd sonar-rnd / SonarQube Code Analysis

vault-client/src/test/java/org/talend/sdk/components/vault/client/VaultClientTest.java#L362

Rename "client" which hides the field declared at line 52.

Check warning on line 398 in vault-client/src/test/java/org/talend/sdk/components/vault/client/VaultClientTest.java

See this annotation in the file changed.

@sonar-rnd sonar-rnd / SonarQube Code Analysis

vault-client/src/test/java/org/talend/sdk/components/vault/client/VaultClientTest.java#L398

Rename "client" which hides the field declared at line 52.