Skip to content

Commit 7bba39e

Browse files
committed
Added constructor validations to prevent NPEs
1 parent c67535c commit 7bba39e

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

vault/src/main/java/io/scalecube/security/vault/VaultServiceRolesInstaller.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -50,15 +50,18 @@ public class VaultServiceRolesInstaller {
5050
private final TimeUnit timeUnit;
5151

5252
private VaultServiceRolesInstaller(Builder builder) {
53-
this.vaultAddress = builder.vaultAddress;
54-
this.vaultTokenSupplier = builder.vaultTokenSupplier;
55-
this.keyNameSupplier = builder.keyNameSupplier;
56-
this.roleNameBuilder = builder.roleNameBuilder;
57-
this.serviceRolesSources = builder.serviceRolesSources;
58-
this.keyAlgorithm = builder.keyAlgorithm;
59-
this.keyRotationPeriod = builder.keyRotationPeriod;
60-
this.keyVerificationTtl = builder.keyVerificationTtl;
61-
this.roleTtl = builder.roleTtl;
53+
this.vaultAddress = Objects.requireNonNull(builder.vaultAddress, "vaultAddress");
54+
this.vaultTokenSupplier =
55+
Objects.requireNonNull(builder.vaultTokenSupplier, "vaultTokenSupplier");
56+
this.keyNameSupplier = Objects.requireNonNull(builder.keyNameSupplier, "keyNameSupplier");
57+
this.roleNameBuilder = Objects.requireNonNull(builder.roleNameBuilder, "roleNameBuilder");
58+
this.serviceRolesSources =
59+
Objects.requireNonNull(builder.serviceRolesSources, "serviceRolesSources");
60+
this.keyAlgorithm = Objects.requireNonNull(builder.keyAlgorithm, "keyAlgorithm");
61+
this.keyRotationPeriod = Objects.requireNonNull(builder.keyRotationPeriod, "keyRotationPeriod");
62+
this.keyVerificationTtl =
63+
Objects.requireNonNull(builder.keyVerificationTtl, "keyVerificationTtl");
64+
this.roleTtl = Objects.requireNonNull(builder.roleTtl, "roleTtl");
6265
this.timeout = builder.timeout;
6366
this.timeUnit = builder.timeUnit;
6467
}

0 commit comments

Comments
 (0)