search

jp-gouin / helm-openldap

Helm chart of Openldap in High availability with multi-master replication and PhpLdapAdmin and Ltb-Passwd
Apache License 2.0
183 stars 115 forks source link

change base dn from dc to o,c #96

Closed ljjolic99 closed 1 year ago

ljjolic99 commented 1 year ago

we need to change the base dn. Our dn is based on ou=xxx,o=yy,c=ca instead of dc=xxx,dc=yyy, how do we go about that

jp-gouin commented 1 year ago

Hi @ljjolic99

Today it's not possible , but to support it it require minimal development Adding a variable in the values to allow override of the ldap domain and use it in the helper when defining the global.baseDomain variable

Would you try to submit a PR ?

jp-gouin commented 1 year ago

fixed in the last version 4.1.0 global.ldapDomain can either be explicit ou=xxx,o=yy,c=ca or domain based example.org which will be translated to dc=example,dc=org

ljjolic99 commented 1 year ago

so i got your latest master and helm install. I can login with "cn=admin,ou=myOrg,o=gc,c=ca" with phpldapadmin but the base DN is not recognized.

phpadminldap gives the "This base cannot be created with PLA." error

my values.yaml:

//imagePullSecrets: [""] // ldapDomain , can be explicit (e.g dc=toto,c=ca) or domain based (e.g example.com) ldapDomain: "ou=myOrg,o=gc,c=ca" // Specifies an existing secret to be used for admin and config user passwords. The expected key are LDAP_ADMIN_PASSWORD and LDAP_CONFIG_ADMIN_PASSWORD. // existingSecret: "" // Default Passwords to use, stored as a secret. Not used if existingSecret is set. adminPassword: adminPassword configPassword: adminPassword

jp-gouin commented 1 year ago

Release will be out in the couple of minute, sorry about that

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.