Miscellaneous first checks
Affected Component(s)
Relaying emails
What happened and when does this occur?
Commit fe21fe7 might have broken the relay host support when using only /tmp/docker-mailserver/postfix-sasl-password.cf.
The condition in line 65 of target/scripts/helpers/relay.sh before was:
if [[ ! -f /tmp/docker-mailserver/postfix-sasl-password.cf ]] \
&& [[ -z ${RELAY_USER} || -z ${RELAY_PASSWORD} ]] \
&& [[ -z ${SASL_PASSWD} ]]
Now the condition is:
if [[ ! -f /tmp/docker-mailserver/postfix-sasl-password.cf ]] \
&& [[ -z ${RELAY_USER} ]] || [[ -z ${RELAY_PASSWORD} ]]
The problem is that this condition seems to be only non-satisfiable if RELAY_PASSWORD set to a nonempty value - regardless of whether the user only wants to use the data from postfix-sasl-password.cf
It appears that this is a bug as the related tickets do not specify this kind of change.
What did you expect to happen?
I would suggest to fix the condition to preserve the backwards compatibility, e.g.:
if [[ ! -f /tmp/docker-mailserver/postfix-sasl-password.cf ]] \
&& [[ -z ${RELAY_USER} || -z ${RELAY_PASSWORD} ]]
How do we replicate the issue?
- Startup the container with
RELAY_HOST=any-value and postfix-sasl-password.cf existing
- "Missing relay-host mapped credentials provided via ENV, or from postfix-sasl-password.cf" is being printed on stdout.
DMS version
master
What operating system is DMS running on?
Linux
Which operating system version?
Debian 11
What instruction set architecture is DMS running on?
AMD64 / x86_64
What container orchestration tool are you using?
Docker
docker-compose.yml
No response
Relevant log output
No response
Other relevant information
No response
What level of experience do you have with Docker and mail servers?
Code of conduct
Improvements to this form?
No response
Miscellaneous first checks
Affected Component(s)
Relaying emails
What happened and when does this occur?
Commit fe21fe7 might have broken the relay host support when using only
/tmp/docker-mailserver/postfix-sasl-password.cf.The condition in line 65 of
target/scripts/helpers/relay.shbefore was:Now the condition is:
The problem is that this condition seems to be only non-satisfiable if
RELAY_PASSWORDset to a nonempty value - regardless of whether the user only wants to use the data frompostfix-sasl-password.cfIt appears that this is a bug as the related tickets do not specify this kind of change.
What did you expect to happen?
I would suggest to fix the condition to preserve the backwards compatibility, e.g.:
How do we replicate the issue?
RELAY_HOST=any-valueandpostfix-sasl-password.cfexistingDMS version
master
What operating system is DMS running on?
Linux
Which operating system version?
Debian 11
What instruction set architecture is DMS running on?
AMD64 / x86_64
What container orchestration tool are you using?
Docker
docker-compose.yml
No response
Relevant log output
No response
Other relevant information
No response
What level of experience do you have with Docker and mail servers?
Code of conduct
Improvements to this form?
No response