Skip to content

Commit c67535c

Browse files
committed
Added constructor validations to prevent NPEs
1 parent 69a6d4d commit c67535c

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

tokens/src/main/java/io/scalecube/security/tokens/jwt/JsonwebtokenResolver.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import io.jsonwebtoken.Jwts;
44
import io.jsonwebtoken.Locator;
55
import java.security.Key;
6+
import java.util.Objects;
67
import java.util.concurrent.CompletableFuture;
78
import org.slf4j.Logger;
89
import org.slf4j.LoggerFactory;
@@ -14,7 +15,7 @@ public class JsonwebtokenResolver implements JwtTokenResolver {
1415
private final Locator<Key> keyLocator;
1516

1617
public JsonwebtokenResolver(Locator<Key> keyLocator) {
17-
this.keyLocator = keyLocator;
18+
this.keyLocator = Objects.requireNonNull(keyLocator, "keyLocator");
1819
}
1920

2021
@Override

tokens/src/main/java/io/scalecube/security/tokens/jwt/JwksKeyLocator.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import java.time.Duration;
2626
import java.util.Base64;
2727
import java.util.Map;
28+
import java.util.Objects;
2829
import java.util.concurrent.ConcurrentHashMap;
2930
import java.util.concurrent.locks.ReentrantLock;
3031

@@ -41,9 +42,9 @@ public class JwksKeyLocator extends LocatorAdapter<Key> {
4142
private final ReentrantLock cleanupLock = new ReentrantLock();
4243

4344
private JwksKeyLocator(Builder builder) {
44-
this.jwksUri = builder.jwksUri;
45-
this.connectTimeout = builder.connectTimeout;
46-
this.requestTimeout = builder.requestTimeout;
45+
this.jwksUri = Objects.requireNonNull(builder.jwksUri, "jwksUri");
46+
this.connectTimeout = Objects.requireNonNull(builder.connectTimeout, "connectTimeout");
47+
this.requestTimeout = Objects.requireNonNull(builder.requestTimeout, "requestTimeout");
4748
this.keyTtl = builder.keyTtl;
4849
}
4950

0 commit comments

Comments
 (0)