TLS to ldap seems to be enforced in the chart

[flixman]

Describe the bug
I am trying to get phpldapadmin to communicate without TLS to openldap. With a clean installation of the chart, the custom value for the variable PHPLDAPADMIN_LDAP_HOSTS is ignored.

To Reproduce
Install the helm chart with the following custom values:

phpldapadmin:
  enabled: true
  ingress: 
    enabled: true
    hosts:
      - phpldapadmin.test.local
  env:
    PHPLDAPADMIN_LDAP_HOSTS: "#PYTHON2BASH:[{ 'openldap.openldap'  : [{'server': [{'tls': False},{'port':389}]},{'login': [{'bind_id': 'cn=openldap,dc=test,dc=local}]}]}]"

Expected behavior
The config map for phpldapadmin is updated with the correct value.

[jp-gouin]

Hi @flixman ,
Yes it is enforced and cannot be overwritten. -> https://github.com/jp-gouin/helm-openldap/blob/master/charts/phpldapadmin/templates/configmap.yaml

You can submit a PR if you want 😀

[flixman]

well... the first question is: is this something that should be supported? I am learning about helm, kubernetes, etc., and I find being able to disable TLS interesting because then I can sniff the network and understand what's going on... however, I do not know if this is something desired or not. Maybe this could be just added to the README?

[jp-gouin]

For a production environment no but for a testing scenario , why not

[stale]

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.