Skip to content

fix(sdk): DSPX-1561 parse manifests strictly - #436

Open
dmihalcik-virtru wants to merge 1 commit into
mainfrom
DSPX-1561-strict-manifest
Open

dmihalcik-virtru wants to merge 1 commit into
mainfrom
DSPX-1561-strict-manifest

fix(sdk): DSPX-1561 parse manifests strictly

9b19e0d
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Nov 9, 2026 in 26s

Quality Gate failed

Failed conditions
D 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 584 in sdk/src/main/java/io/opentdf/platform/sdk/SDK.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This class has 7 parents which is greater than 5 authorized.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzf_H4KxBVUKC8zPE&open=AaHAzf_H4KxBVUKC8zPE&pullRequest=436

Check warning on line 1621 in sdk/src/test/java/io/opentdf/platform/sdk/TDFTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzf7d4KxBVUKC8zPD&open=AaHAzf7d4KxBVUKC8zPD&pullRequest=436

Check warning on line 640 in sdk/src/main/java/io/opentdf/platform/sdk/SDK.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=opentdf_java-sdk&issues=AaHAzf_H4KxBVUKC8zPF&open=AaHAzf_H4KxBVUKC8zPF&pullRequest=436

Check warning on line 444 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPR&open=AaHAzgB34KxBVUKC8zPR&pullRequest=436

Check warning on line 492 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.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=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPG&open=AaHAzgB34KxBVUKC8zPG&pullRequest=436

Check warning on line 516 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.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=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPI&open=AaHAzgB34KxBVUKC8zPI&pullRequest=436

Check failure on line 61 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPJ&open=AaHAzgB34KxBVUKC8zPJ&pullRequest=436

Check failure on line 425 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPO&open=AaHAzgB34KxBVUKC8zPO&pullRequest=436

Check failure on line 61 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPK&open=AaHAzgB34KxBVUKC8zPK&pullRequest=436

Check warning on line 429 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPP&open=AaHAzgB34KxBVUKC8zPP&pullRequest=436

Check warning on line 432 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPQ&open=AaHAzgB34KxBVUKC8zPQ&pullRequest=436

Check warning on line 511 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.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=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPH&open=AaHAzgB34KxBVUKC8zPH&pullRequest=436

Check failure on line 63 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPL&open=AaHAzgB34KxBVUKC8zPL&pullRequest=436

Check warning on line 244 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPM&open=AaHAzgB34KxBVUKC8zPM&pullRequest=436

Check warning on line 136 in sdk/src/test/java/io/opentdf/platform/sdk/StrictManifestParsingTest.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=opentdf_java-sdk&issues=AaHAzf0V4KxBVUKC8zPC&open=AaHAzf0V4KxBVUKC8zPC&pullRequest=436

Check failure on line 411 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix this access on a collection that will throw an 'ArrayIndexOutOfBoundsException' when executed.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPS&open=AaHAzgB34KxBVUKC8zPS&pullRequest=436

Check failure on line 318 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.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=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPN&open=AaHAzgB34KxBVUKC8zPN&pullRequest=436

Check warning on line 110 in sdk/src/test/java/io/opentdf/platform/sdk/StrictManifestParsingTest.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=opentdf_java-sdk&issues=AaHAzf0V4KxBVUKC8zPB&open=AaHAzf0V4KxBVUKC8zPB&pullRequest=436

Check failure on line 284 in sdk/src/main/java/io/opentdf/platform/sdk/StrictJsonReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix this access that will throw a NullPointerException when executed.

See more on https://sonarcloud.io/project/issues?id=opentdf_java-sdk&issues=AaHAzgB34KxBVUKC8zPT&open=AaHAzgB34KxBVUKC8zPT&pullRequest=436