Add missing null check to tag translator method (#69)

* Add missing null check to tag translator method * Move check to handler to avoid ambiguous code * Remove erroneous contract-test-artifacts
aws-cloudformation · Sep 23, 2024 · 28aac84 · 28aac84
1 parent ba8ad1f
commit 28aac84
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 224 deletions.
diff --git a/aws-transfer-server/src/main/java/software/amazon/transfer/server/CreateHandler.java b/aws-transfer-server/src/main/java/software/amazon/transfer/server/CreateHandler.java
@@ -81,7 +81,7 @@ private CreateServerRequest translateToCreateRequest(final ResourceModel model)
                 .protocols(translateToSdkProtocols(model.getProtocols()))
                 .protocolDetails(ProtocolDetailsTranslator.toSdk(model.getProtocolDetails()))
                 .securityPolicyName(model.getSecurityPolicyName())
-                .tags(translateToSdkTags(model.getTags()))
+                .tags(CollectionUtils.isNullOrEmpty(model.getTags()) ? null : translateToSdkTags(model.getTags()))
                 .workflowDetails(WorkflowDetailsTranslator.toSdk(model.getWorkflowDetails(), false))
                 .structuredLogDestinations(model.getStructuredLogDestinations())
                 .s3StorageOptions(S3StorageOptionsTranslator.toSdk(model.getS3StorageOptions()))

diff --git a/aws-transfer-workflow/contract-tests-artifacts/dependencies.yml b/aws-transfer-workflow/contract-tests-artifacts/dependencies.yml
diff --git a/aws-transfer-workflow/contract-tests-artifacts/inputs_1.json b/aws-transfer-workflow/contract-tests-artifacts/inputs_1.json