diff options
Diffstat (limited to 'jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java')
-rw-r--r-- | jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java b/jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java index 61e0051022d..f2cac68c030 100644 --- a/jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java +++ b/jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java @@ -96,11 +96,11 @@ public class AwsParameterStore extends AbstractComponent implements SecretStore String region; AwsSettings(String name, String role, String awsId, String externalId, String region) { - this.name = name; - this.role = role; - this.awsId = awsId; - this.externalId = externalId; - this.region = region; + this.name = validate(name, "name"); + this.role = validate(role, "role"); + this.awsId = validate(awsId, "awsId"); + this.externalId = validate(externalId, "externalId"); + this.region = validate(region, "region"); } @@ -142,5 +142,11 @@ public class AwsParameterStore extends AbstractComponent implements SecretStore slime.setString("externalId", "*****"); slime.setString("region", region); } + + static String validate(String value, String name) { + if (value == null || value.isBlank()) + throw new IllegalArgumentException("Config parameter '" + name + "' was blank or empty"); + return value; + } } } |