Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I was trying to extend the postfix configuration by adding custom directives to extra.cf, and found out that some of them were disappearing on postfix-mailcow restarts. Specifically, this line:
smtpd_banner = $myhostname ESMTP Postfix
Same problem is described here.
That happens due to untied
/myhostname/d
sed operation indata/Dockerfiles/postfix/postfix.sh
- it deletes not only themyhostname = example.org
line before updating it, but also lines like mentioned above.This PR fixes the issue by ignoring lines that contain
$myhostname
(variable, not a directive) when cleaning up theextra.cf
.