Update dovecot-ldap.conf.ext#1557
Conversation
add auth_bind = no so that it can be overridden via the env-mailserver file used by docker compose. This is related to docker-mailserver#1526
|
As @guardiande mentions in #1526 auth_bind is needed for some setups. Setting a default of no, so it doesn't break the existing setup. But by having the value in the dovecot-ldap.conf.ext it should be overridable with a env variable like DOVECOT_AUTH_BIND. |
|
Is this PR work in progress? You mention an env variable, but it is not there? |
From So creating the container with (that's the variable you're missing, it's not part of the PR 😊) would allow for closing #1526 |
|
Ah, ok. I misunderstood. I didn't think we had to specify all environment variables up front, only if you needed it used in the container. So I didn't add this to the env file, so it would retain the current default behavior. But I can update the env file to include it. |
No, you don't have to include the variable in the env file. You did everything right. Only if someone - like me - wants to have auth_bind set to yes she would have to override the default setting (which is no) to yes by including the |
add environment variable for DOVECOT_AUTH_BIND
Ya, I noticed there were a few other DOVECOT_ variables that could be used, but were not specified (DOVECOT_USER_ATTRS, DOVECOT_PASS_ATTRS, and DOVECOT_DEFAULT_PASS_SCHEME). But since the AUTH_BIND wasn't defined in the dovecot ldap.conf, it couldn't be overridden with environment variable. I just committed another change to add DOVECOT_AUTH_BIND to the env file, so if @erik-wramner wants to review now, I think this can be pulled. |
add auth_bind = no so that it can be overridden via the env-mailserver file used by docker compose.
This is related to #1526