Just be careful not to mess with ldap, this is central part of Yunohost machine.
from my experience:
I wanted to be clever and allow the docker containers to connect to Idap.
So I added in the line ldap://172.17.0.1:389/ instead of commenting. It worked but later I had a hard time. At the next reboot, as the docker interface is mounted after ldap is up, the ldap service was failing and I was locked out of the system (needed direct access to the machine to connect as root)…
Finally I just commented the line and added a special IPtables rule instead.
edit: as mentioned by several posters above, commenting the line creates another kind of mess, admin is unable to access ldap… (with this error: Exception AttributeError: "'Authenticator' object has no attribute 'con'" in <bound method Authenticator.__del__ of <moulinette.authenticators.ldap.Authenticator object at 0x7f5d68adfa50>> ignored.)
I did not try yet but maybe it’s better to put ldap:/// instead.