Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] SpamAssassin dies if not update possible, then amavisd and the mail server collapse #60

Closed
stdevPavelmc opened this issue Sep 18, 2020 · 2 comments
Closed

[BUG] SpamAssassin dies if not update possible, then amavisd and the mail server collapse #60

stdevPavelmc opened this issue Sep 18, 2020 · 2 comments
Assignees
@stdevPavelmc
Labels
bug Something isn't working Urgent

Comments

@stdevPavelmc
Copy link
Owner

Describe the bug
If your setup does not have a working DNS (like a local domain with no upstream DNS) the spamassassin service will die and amavisd-new will die also, breaking the mail flow.

Possible solution

  • Check in conf-check the test stage if we have selected SPAMD the DNS can resolve the TXT register for spamassassin 2.4.3.updates.spamassassin.org it's a CNAME of "3.3.3...." and it return the TXT fingerprint of the database
  • If not working notice the user with an error about it, to fix the error or disable the SPAMD in the config
@stdevPavelmc stdevPavelmc added the bug Something isn't working label Sep 18, 2020
@stdevPavelmc stdevPavelmc self-assigned this Sep 18, 2020
@stdevPavelmc stdevPavelmc mentioned this issue Sep 18, 2020
Closed
@stdevPavelmc
Copy link
Owner Author

After install if the sa-update ran failed you have 48 hours of happines... when the databases go old, bum!

@stdevPavelmc
Copy link
Owner Author

Ok, similar solution to the one in clamav updates, check in the check-conf target, if no TXT reachable for spam-assassin updates, fail if spmad configured

stdevPavelmc added a commit that referenced this issue Sep 21, 2020
@stdevPavelmc
Refs #60, adding the test logic and the error message in the checks
493cd4d
stdevPavelmc added a commit that referenced this issue Sep 21, 2020
@stdevPavelmc
Refs #60, improve the logic on the provision: now we check the config…
a7cf56e 
… on every povision to check for AV and SPAMD DNS fails
stdevPavelmc added a commit that referenced this issue Sep 21, 2020
@stdevPavelmc
Refs #60, add the fixt to the changelog
ef2a107
stdevPavelmc added a commit that referenced this issue Sep 21, 2020
@stdevPavelmc
Refs #60, add a comment about the problem in the FAQs
d6818b1
stdevPavelmc added a commit that referenced this issue Sep 21, 2020
@stdevPavelmc
Refs #60, Cosmetic touches to the README
8b2690a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stdevPavelmc stdevPavelmc

Labels
bug Something isn't working Urgent
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stdevPavelmc