Skip to content

Commit

Permalink
fix : dynamo config optional + remove dev env
Browse files Browse the repository at this point in the history
  • Loading branch information
jecos committed Feb 21, 2022
1 parent ae5db2e commit ac86645
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
ecs_service_type_1_standard {
projectName = "kf-key-manager"
internal_app = "false"
environments = "dev,qa,prd"
environments = "qa,prd"
docker_image_type = "debian"
dependencies = "ecr"
create_default_iam_role = "0"
Expand Down
8 changes: 4 additions & 4 deletions src/main/java/io/kidsfirst/config/DynamoDbConfig.java
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ public class DynamoDbConfig {
private final String accessKey;
private final String secretKey;

public DynamoDbConfig(@Value("${aws.dynamodb.endpoint}") String dynamoDbEndPointUrl,
@Value("${aws.region}") String region,
@Value("${aws.accessKey}") String accessKey,
@Value("${aws.secretKey}") String secretKey) {
public DynamoDbConfig(@Value("${aws.dynamodb.endpoint#{null}}") String dynamoDbEndPointUrl,
@Value("${aws.region#{null}}") String region,
@Value("${aws.accessKey#{null}}") String accessKey,
@Value("${aws.secretKey#{null}}") String secretKey) {
this.dynamoDbEndPointUrl = dynamoDbEndPointUrl;
this.region = region;
this.accessKey = accessKey;
Expand Down

0 comments on commit ac86645

Please sign in to comment.