Merge pull request opencloud-eu#85 from kastl-ars/20250613_remove_basicAuth

[opencloud] remove the enableBasicAuth variable, as Basic Auth should not be used

Author: butonic

charts/opencloud/README.md

@@ -248,7 +248,6 @@ This will prepend `my-registry.com/` to all image references in the chart. For e
 | `opencloud.logColor` | Enable log color | `false` |
 | `opencloud.logPretty` | Enable pretty logging | `false` |
 | `opencloud.insecure` | Insecure mode (for self-signed certificates) | `true` |
-| `opencloud.enableBasicAuth` | Enable basic auth | `false` |
 | `opencloud.adminPassword` | Admin password | `admin` |
 | `opencloud.createDemoUsers` | Create demo users | `false` |
 | `opencloud.resources` | CPU/Memory resource requests/limits | `{}` |

charts/opencloud/templates/opencloud/deployment.yaml

@@ -162,7 +162,7 @@ spec:
               value: {{ tpl (toString .Values.opencloud.insecure) . | quote }}
             # Basic auth (only needed when not using Keycloak)
             - name: PROXY_ENABLE_BASIC_AUTH
-              value: {{ tpl (toString .Values.opencloud.enableBasicAuth) . | quote }}
+              value: false
             # These vars are needed to the csp config file to include the web office apps and the importer
             - name: ONLYOFFICE_DOMAIN
               value: "{{ .Values.global.domain.onlyoffice }}"

charts/opencloud/values.yaml

@@ -434,8 +434,6 @@ opencloud:
   logPretty: false
   # Insecure mode (for self-signed certificates)
   insecure: true
-  # Enable basic auth (set to false to use Keycloak only)
-  enableBasicAuth: "{{ not .Values.keycloak.enabled }}"
   # Admin password
   adminPassword: admin
   # Create demo users