Skip to content

Commit

Permalink
do not require .Values.remoteAuth.ldap.bindDn when .Values.remoteAuth…
Browse files Browse the repository at this point in the history
….ldap.serverUri is specified (#508)
  • Loading branch information
0xmeyer committed Jan 21, 2025
1 parent e8c6fe3 commit 626d647
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion charts/netbox/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ netbox: postgresql
Validate values of Netbox Chart - LDAP
*/}}
{{- define "netbox.validateValues.ldap" -}}
{{- if and (has "netbox.authentication.LDAPBackend" .Values.remoteAuth.backends) (or (empty .Values.remoteAuth.ldap.serverUri) (empty .Values.remoteAuth.ldap.bindDn)) -}}
{{- if and (has "netbox.authentication.LDAPBackend" .Values.remoteAuth.backends) (and (empty .Values.remoteAuth.ldap.serverUri) (empty .Values.remoteAuth.ldap.bindDn)) -}}
netbox: remoteAuth.ldap
When LDAP backend is activated, you must provide all the necessary parameters.
Review the values under `remoteAuth.ldap`.
Expand Down

0 comments on commit 626d647

Please sign in to comment.