Merge pull request #2610 from ASFHyP3/edc-s3-read

quote cloudformation parameter overrides that are deployment-dependent
ASFHyP3 · Feb 20, 2025 · b8ab6b0 · b8ab6b0
2 parents 849060c + e09c579
commit b8ab6b0
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/actions/deploy-hyp3/action.yml b/.github/actions/deploy-hyp3/action.yml
@@ -94,16 +94,16 @@ runs:
             || export ROLE_STATEMENT="--role-arn ${{ inputs.CLOUDFORMATION_ROLE_ARN }}"
 
           [ -z ${{ inputs.DOMAIN_NAME }} ] && export DOMAIN_NAME="" \
-            || export DOMAIN_NAME="DomainName=${{ inputs.DOMAIN_NAME }}"
+            || export DOMAIN_NAME="DomainName='${{ inputs.DOMAIN_NAME }}'"
 
           [ -z ${{ inputs.CERTIFICATE_ARN }} ] && export CERTIFICATE_ARN="" \
-            || export CERTIFICATE_ARN="CertificateArn=${{ inputs.CERTIFICATE_ARN }}"
+            || export CERTIFICATE_ARN="CertificateArn='${{ inputs.CERTIFICATE_ARN }}'"
 
           [ -z ${{ inputs.BUCKET_READ_PRINCIPALS }} ] && export BUCKET_READ_PRINCIPALS="" \
-            || export BUCKET_READ_PRINCIPALS="BucketReadPrincipals=${{ inputs.BUCKET_READ_PRINCIPALS }}"
+            || export BUCKET_READ_PRINCIPALS="BucketReadPrincipals='${{ inputs.BUCKET_READ_PRINCIPALS }}'"
 
           [ -z ${{ inputs.DISTRIBUTION_URL }} ] && export DISTRIBUTION_URL="" \
-            || export DISTRIBUTION_URL="DistributionUrl=${{ inputs.DISTRIBUTION_URL }}"
+            || export DISTRIBUTION_URL="DistributionUrl='${{ inputs.DISTRIBUTION_URL }}'"
 
           aws cloudformation package \
             --template-file apps/main-cf.yml \