Release share chart v1.1.1 (#371)

Alfresco · Sep 25, 2024 · be66bd4 · be66bd4
1 parent 79f16b5
commit be66bd4
Show file tree

Hide file tree

Showing 5 changed files with 38 additions and 22 deletions.
diff --git a/charts/alfresco-share/Chart.lock b/charts/alfresco-share/Chart.lock
@@ -1,6 +1,6 @@
 dependencies:
 - name: alfresco-common
   repository: https://alfresco.github.io/alfresco-helm-charts
-  version: 3.1.3
-digest: sha256:0bf5dfbbb9c0c77dafb7ec3b4ba3b3a39907b59704fe9193bb5788a67184ac90
-generated: "2024-06-17T18:10:07.20883+02:00"
+  version: 3.1.4
+digest: sha256:219597e107a944840c8a219b4248d7dd3ccd9fa9138e6c052c36b3a7cc13645a
+generated: "2024-09-23T14:26:53.407175+02:00"
diff --git a/charts/alfresco-share/Chart.yaml b/charts/alfresco-share/Chart.yaml
@@ -2,9 +2,9 @@ apiVersion: v2
 name: alfresco-share
 description: Alfresco Share Helm chart for Kubernetes
 type: application
-version: 1.1.0
+version: 1.1.1
 appVersion: 23.3.0
 dependencies:
   - repository: https://alfresco.github.io/alfresco-helm-charts
-    version: 3.1.3
+    version: 3.1.4
     name: alfresco-common
diff --git a/charts/alfresco-share/README.md b/charts/alfresco-share/README.md
@@ -5,7 +5,7 @@ parent: Charts Reference
 
 # alfresco-share
 
-![Version: 1.1.0](https://img.shields.io/badge/Version-1.1.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 23.3.0](https://img.shields.io/badge/AppVersion-23.3.0-informational?style=flat-square)
+![Version: 1.1.1](https://img.shields.io/badge/Version-1.1.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 23.3.0](https://img.shields.io/badge/AppVersion-23.3.0-informational?style=flat-square)
 
 Alfresco Share Helm chart for Kubernetes
 
@@ -15,7 +15,7 @@ Checkout [alfresco-content-services chart's doc](https://github.com/Alfresco/acs
 
 | Repository | Name | Version |
 |------------|------|---------|
-| https://alfresco.github.io/alfresco-helm-charts | alfresco-common | 3.1.3 |
+| https://alfresco.github.io/alfresco-helm-charts | alfresco-common | 3.1.4 |
 
 ## Values
 

diff --git a/charts/alfresco-share/ci/default-values.yaml b/charts/alfresco-share/ci/default-values.yaml
@@ -2,3 +2,7 @@ tags:
   ci: true
 
 replicaCount: 2
+
+known_urls:
+  - "http://localhost:8080"
+  - "http://localhost:8080/share"
diff --git a/charts/alfresco-share/tests/deployment_test.yaml b/charts/alfresco-share/tests/deployment_test.yaml
@@ -131,36 +131,48 @@ tests:
             provided known_urls MUST start with a scheme (http :// or https://)
   - it: |
       should render custom CSRF config.
-      referers MUST be a regex with pipes escaped (due to the way vars are substitued in Share pods
-      origins MUST be a list of hosts (without paths) separated by commas.
+      referrers MUST be a regex with pipes escaped (due to the way vars are substituted in Share pods
+      origins MUST be a list of hosts (without paths) separated by pipes.
     set:
       known_urls:
         - https://ecm.domain.tld/myapp/callback
         - https://ecm.domain.tld/alfresco
         - http://app.domain.local:8000
     asserts:
       - equal:
-          path: spec.template.spec.containers[0].env[0].value
-          value: >-
-            https://ecm.domain.tld/myapp/callback/.*\|https://ecm.domain.tld/alfresco/.*\|http://app.domain.local:8000/.*
+          path: spec.template.spec.containers[0].env[0]
+          value:
+            name: CSRF_FILTER_REFERER
+            value: >-
+              https://ecm.domain.tld/myapp/callback/.*\|https://ecm.domain.tld/alfresco/.*\|http://app.domain.local:8000/.*
       - equal:
-          path: spec.template.spec.containers[0].env[1].value
-          value: https://ecm.domain.tld,https://ecm.domain.tld,http://app.domain.local:8000
+          path: spec.template.spec.containers[0].env[1]
+          value:
+            name: CSRF_FILTER_ORIGIN
+            value: https://ecm.domain.tld\|https://ecm.domain.tld\|http://app.domain.local:8000
       - equal:
-          path: spec.template.spec.containers[0].env[2].value
-          value: https://ecm.domain.tld
+          path: spec.template.spec.containers[0].env[2]
+          value:
+            name: BASEURL
+            value: https://ecm.domain.tld
 
   - it: should render a default Xorigin restriction config
     asserts:
       - equal:
-          path: spec.template.spec.containers[0].env[0].value
-          value: http://localhost/.*\|https://localhost/.*
+          path: spec.template.spec.containers[0].env[0]
+          value:
+            name: CSRF_FILTER_REFERER
+            value: http://localhost/.*\|https://localhost/.*
       - equal:
-          path: spec.template.spec.containers[0].env[1].value
-          value: http://localhost,https://localhost
+          path: spec.template.spec.containers[0].env[1]
+          value:
+            name: CSRF_FILTER_ORIGIN
+            value: http://localhost\|https://localhost
       - equal:
-          path: spec.template.spec.containers[0].env[2].value
-          value: http://localhost
+          path: spec.template.spec.containers[0].env[2]
+          value:
+            name: BASEURL
+            value: http://localhost
 
   - it: should render extra configs
     set: