Merge pull request #180 from xenit-eu/KANSUP74-37

KANSUP74-37 uncouple inbound mail port on acs-service and -pod

Author: RVanhuysseXenit

CHANGELOG.md

@@ -6,6 +6,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 chronology things are added/fixed/changed and - where possible - links to the PRs involved.
 
 ### Changes
+[v0.8.11]
+* uncouple port for inbound email on acs pod and acs service
+
 [v0.8.10]
 * Make port for inbound email on acs configurable
 

README.md

@@ -483,8 +483,17 @@ ingress:
 
 * Required: false
 * Default: None
-* Description: Set given port for inbound mail in the acs config and expose it in the acs-service & deployment. 
-* **Warning**: Setting only this port does not enable the system. Enabling requires additional acs config (see alfresco documentation) and configuration of the ingress (see applicable ingress documentation)
+* Description: Set given port for inbound mail in the acs config and deployment
+* **Warning**: Setting only this port does not enable the system. Enabling requires additional acs config (see alfresco documentation).
+* **Warning**: Modern acs images do not run the application as root, which precludes it from binding on priviliged ports -like the smtp well-known port 25.
+* **Warning**: Setting only this property does not expose the port on the acs-service or the ingress. This freatures requires the property `acs.email.inbound.servicePort` and additional configuration of the ingress (see applicable ingress documentation)
+
+#### `acs.email.inbound.servicePort`
+
+* Required: false
+* Default: None
+* Description: Set given port for inbound mail in the acs service
+* **Warning**: Setting only this port does not enable the system. Enabling requires additional config (see above `acs.email.inbound.port`)
 
 #### `acs.additionalEnvironmentVariables`
 

xenit-alfresco/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.8.10
+version: 0.8.11
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

xenit-alfresco/templates/acs/acs-service.yaml

@@ -20,10 +20,11 @@ spec:
         protocol: TCP
         port: 8443
         targetPort: 8443
-    {{- if ((((.Values.acs).email).inbound).port) }}
+    {{- if and ((((.Values.acs).email).inbound).servicePort) ((((.Values.acs).email).inbound).port) }}
       - name: 'acs-inbound-smtp'
         protocol: TCP
-        port: {{ .Values.acs.email.inbound.port }}
+        port: {{ .Values.acs.email.inbound.servicePort }}
+        targetPort:  {{ .Values.acs.email.inbound.port }}
     {{- end }}
   {{- if .Values.general.serviceType }}
   type: {{ .Values.general.serviceType }}