Merge pull request #2 from CodeTogether-Inc/hide-clients-endpoint

Optionally hide clients endpoint
CodeTogether-Inc · Jan 9, 2024 · 60cb0fc · 60cb0fc
2 parents 958f098 + c0f1bce
commit 60cb0fc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/charts/live/templates/deployment.yaml b/charts/live/templates/deployment.yaml
@@ -61,6 +61,11 @@ spec:
           {{- end }}
           {{- end }}
 
+          {{- if eq .Values.codetogether.noclients "true" }}
+          - name: CT_NOCLIENTS
+            value: "true"
+          {{- end }}
+
           {{- if eq .Values.codetogether.mode "locator-central" }}
           - name: CT_LOCATOR
             value: {{ .Values.codetogether.url | quote }}

diff --git a/charts/live/values.yaml b/charts/live/values.yaml
@@ -63,6 +63,8 @@ codetogether:
   # HTTPS is required. If your cert is not a fully-rooted (trusted cert), set
   # to true to allow more tolerance from clients
   trustAllCerts: "true"
+  # Optionally do not expose /clients endpoint by setting to 'true'
+  noclients: "false"
   # Set a customized TZ for CodeTogether container (default is UTC)
   # https://nodatime.org/TimeZones
   timeZone: