aboutsummaryrefslogtreecommitdiffstats
path: root/jdisc-cloud-aws/src/main/java/com/yahoo/jdisc/cloud/aws/AwsParameterStore.java
diff options
context:
space:
mode:
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.java16
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;
+ }
}
}