-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Amavis should reload config for /etc/postfix/vhost
updates
#2616
fix: Amavis should reload config for /etc/postfix/vhost
updates
#2616
Conversation
Amavis was not aware of new domains in vhost as it did not refresh it's sources upon change detection.
25ffbfc
to
aa873c7
Compare
then | ||
# /etc/postfix/vhost was updated, amavis must refresh it's config by | ||
# reading this file again in case of new domains, otherwise they will be ignored. | ||
amavisd-new reload |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Presumably supervisorctl restart amavis
should be used.
I've not received any feedback about better understanding why postfix reload
was an issue in the past (may have been due to the present postfix-wrapper.sh
script?).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Truth be told, I don‘t know 🙈 @casperklein do you have an idea?:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The usage of amavisd-new reload
should be fine. However, if for some reason a complete service restart is needed, supervisorctl restart amavis
should be used.
Description
When
/etc/postfix/vhost
is updated from change detection, Amavis did not refresh that source, and thus would not be aware of any new domains, ignoring them for spam/virus checking.A test could probably be added, my time is focused on prepping PRs with the time I have available right now.
Type of change
Checklist: