Skip to content

Commit 42bcabf

Browse files
(Sonar) Fixed finding: "Classes should not be loaded dynamically"
1 parent 3cc8295 commit 42bcabf

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

pom.xml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@
1212
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
1313
<sonar.host.url>https://sonarcloud.io</sonar.host.url>
1414
<sonar.organization>pixee</sonar.organization>
15-
</properties>
15+
<versions.java-security-toolkit>1.2.1</versions.java-security-toolkit>
16+
</properties>
1617

1718
<dependencyManagement>
1819
<dependencies>
@@ -31,7 +32,7 @@
3132
<dependency>
3233
<groupId>io.github.pixee</groupId>
3334
<artifactId>java-security-toolkit</artifactId>
34-
<version>1.2.0</version>
35+
<version>${versions.java-security-toolkit}</version>
3536
</dependency>
3637
</dependencies>
3738
</dependencyManagement>

src/main/java/com/acme/reflection/UnsafeReflection.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package com.acme.reflection;
22

3+
import io.github.pixee.security.Reflection;
34
import jakarta.ws.rs.GET;
45
import jakarta.ws.rs.Path;
56
import jakarta.ws.rs.QueryParam;
@@ -22,7 +23,7 @@ public String hello(@QueryParam("translator") final String translationStrategy)
2223
private static TranslatorStrategy loadTranslatorByName(final String translationStrategy) {
2324
final Class<?> translatorClazz;
2425
try {
25-
translatorClazz = Class.forName("com.acme." + translationStrategy);
26+
translatorClazz = Reflection.loadAndVerify("com.acme." + translationStrategy);
2627
} catch (ClassNotFoundException e) {
2728
throw new IllegalArgumentException("Invalid translator: " + translationStrategy, e);
2829
}

0 commit comments

Comments
 (0)