do not specify LDAP_BASEDN in dockerfile

ahaenggli / AzureAD-LDAP-wrapper

LDAP-Wrapper for 'microsoft 365' work or school accounts/users (former 'office 365' - via Entra ID, former AzureAD without AADDS)

https://ahaenggli.github.io/AzureAD-LDAP-wrapper/

MIT License

121 stars 30 forks source link

do not specify LDAP_BASEDN in dockerfile #60

Closed yufukui-m closed 1 year ago

yufukui-m commented 1 year ago

this config value is generated from env var LDAP_DOMAIN automatically, but the default value in Dockerfile is used because it is specified in the Dockerfile ( also affects LDAP_GROUPSDN, LDAP_USERSDN, LDAP_USERSGROUPSBASEDN and LDAP_SAMBADOMAINNAME )

ahaenggli commented 1 year ago

The value was not always generated, so the variable seems to have remained in the Dockerfile. I will be happy to adjust this with one of the next versions.