changed oidc to openid-connect

defenseunicorns · Jul 25, 2024 · 004b819 · 004b819
1 parent 02ac5a6
commit 004b819
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/bundle/uds-bundle.yaml b/bundle/uds-bundle.yaml
@@ -20,5 +20,5 @@ packages:
               description: "Boolean to enable or disable sso things"
               path: "sso.enabled"
             - name: JENKINS_SSO_PROTOCOL
-              description: "Protocol to use. Valid values are 'oidc' and 'saml'. Default value is 'saml'"
+              description: "Protocol to use. Valid values are 'openid-connect' and 'saml'. Default value is 'saml'"
               path: "sso.protocol"
diff --git a/chart/templates/uds-package.yaml b/chart/templates/uds-package.yaml
@@ -37,14 +37,16 @@ spec:
                         idpMetadataConfiguration:
                           period: 1440
                           url: "https://sso.{{ .Values.domain }}/realms/uds/protocol/saml/descriptor"
-    {{- else if eq .Values.sso.protocol "oidc"  }}
-      clientId: uds-package-jenkins-oidc
+    {{- else if eq .Values.sso.protocol "openid-connect"  }}
+      clientId: uds-package-jenkins-openid-connect
+
+      protocol: openid-connect
 
       secretName: {{ .Values.sso.secretName }}
       secretTemplate:
         sso: |-
           oic:
-                        clientId: "uds-package-jenkins-oidc"
+                        clientId: "uds-package-jenkins-openid-connect"
                         clientSecret: "clientField(secret)"
                         wellKnownOpenIDConfigurationUrl: "https://sso.{{ .Values.domain }}/realms/uds"
                         tokenServerUrl: "https://sso.{{ .Values.domain }}/realms/uds/protocol/openid-connect/token"

diff --git a/chart/values.yaml b/chart/values.yaml
@@ -3,7 +3,7 @@ domain: "###ZARF_VAR_DOMAIN###"
 sso:
   enabled: true
   secretName: jenkins-sso
-  # protocol should be either 'saml' or 'oidc'
+  # protocol should be either 'saml' or 'openid-connect'
   protocol: saml
 
 # customNetworkPolicies: