search

georchestra / docker

Quick start geOrchestra with docker
17 stars 22 forks source link

Extra LDAPHOST env var for LDAP service #223

Open fvanderbiest opened 1 year ago

fvanderbiest commented 1 year ago

What is it for @jeanmi151 ? https://github.com/georchestra/docker/blob/b4b553a98bdd9a442001b995352a282d0c1982c5/docker-compose.yml#L74

jeanmi151 commented 1 year ago

something forgotten ... will remove it

jeanmi151 commented 1 year ago

huuuum aaa no, it is for saying that for the container ldap, ldaphost is the localhost, otherwise it will try to contact it (and it will not work), If I remember correctly , will test it

jeanmi151 commented 1 year ago

Yes, the container fail to start if LDAPHOST if not set at "localhost", I will add a comment

jeanmi151 commented 1 year ago

https://github.com/georchestra/docker/commit/9aeb4babbd3302e31d6bd4b04de34e448d9741a1 , will cherry-pik on master too

jeanmi151 commented 1 year ago

https://github.com/georchestra/docker/pull/224