Merge pull request IQSS#164 from GlobalDataverseCommunityConsortium/1…

…63_cors_bypass allow cors step bypass
dsd-sztaki-hu · Jul 28, 2021 · 944640c · 944640c
2 parents 0660fd2 + 429253e
commit 944640c
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 4 deletions.
diff --git a/defaults/main.yml b/defaults/main.yml
@@ -247,6 +247,7 @@ s3:
   access_key: 4cc355_k3y
   secret_access_key: s3cr3t_4cc355_k3y
   bucket_name: s3-test
+  cors_already_set: false
   create_bucket: true
   # for non-amazon services. for localstack, you might set
   #custom_endpoint_url: "http://localhost:4572"

diff --git a/tasks/s3.yml b/tasks/s3.yml
@@ -77,8 +77,8 @@
     owner: '{{ dataverse.payara.user }}'
     group: '{{ dataverse.payara.group }}'
     mode: 0644
-  when: s3.download_redirect == true or
-    s3.upload_redirect == true
+  when: (s3.download_redirect == true or s3.upload_redirect == true)
+    and s3.cors_already_set == false
 
 - name: enable CORS on S3 bucket
   shell:
@@ -89,9 +89,9 @@
   become_user: '{{ dataverse.payara.user }}'
   environment:
     PATH: "{{ lookup('env', 'PATH') }}:/usr/local/bin"
-  when: (s3.download_redirect == true or
-    s3.upload_redirect == true)
+  when: (s3.download_redirect == true or s3.upload_redirect == true)
     and custom_endpoint_url | length == 0
+    and s3.cors_already_set == false
 
 - include: s3_custom_endpoint_url.yml
   when: custom_endpoint_url | length > 0

diff --git a/tests/group_vars/jenkins.yml b/tests/group_vars/jenkins.yml
@@ -241,6 +241,7 @@ s3:
   access_key: 4cc355_k3y
   secret_access_key: s3cr3t_4cc355_k3y
   bucket_name: s3-test
+  cors_already_set: false
   create_bucket: true
   # for non-amazon services.
   custom_endpoint_url: "http://localhost:4572"

diff --git a/tests/group_vars/memorytests.yml b/tests/group_vars/memorytests.yml
@@ -243,6 +243,7 @@ s3:
   access_key: 4cc355_k3y
   secret_access_key: s3cr3t_4cc355_k3y
   bucket_name: s3-test
+  cors_already_set: false
   create_bucket: true
   # for non-amazon services.
   custom_endpoint_url: "http://localhost:4572"

diff --git a/tests/group_vars/vagrant.yml b/tests/group_vars/vagrant.yml
@@ -234,6 +234,7 @@ s3:
   secret_access_key: s3cr3t_4cc355_k3y
   bucket_name: s3-test
   create_bucket: true
+  cors_already_set: false
   # for non-amazon services. for localstack, you might set
   #custom_endpoint_url: "http://localhost:4572"
   custom_endpoint_url: