Skip to content

rspamd: follow-up of #3016#3039

Merged
georglauterbach merged 5 commits intomasterfrom
rspamd/4
Jan 30, 2023
Merged

rspamd: follow-up of #3016#3039
georglauterbach merged 5 commits intomasterfrom
rspamd/4

Conversation

@georglauterbach
Copy link
Copy Markdown
Member

Description

A small follow-up for things I noticed while getting Rspamd to work today. I should have thought about the permissions issue before, sorry. The disasbled modules were adjusted as well.

I can report though that Rspamd works nicely on my setup now :D

UPDATE: Correction; it works like a charm! It handles DKIM signing now (RSA+ED25519) & DNSBLs properly.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (non-breaking change that does improve existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Rspamd too needs adjustments when it comes to permissions for
`/var/mail-state/`.
1. the DKIM signing module does not need to be explicitly disabled as
   leaving it unconfigured suffices
2. The neural module is pretty useless if not configured and spams the
   log (undesirabled)
3. We do want the RBL module (I will probably make this configurable in
   the future) to be enabled to check DNSBLs by default
This is generally useful to understand the score an email received. It
will not be added to outbound mail for authenticated users / users
connecting from secure networks.
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: db4e4f8

@georglauterbach georglauterbach merged commit 1a0c2a3 into master Jan 30, 2023
@georglauterbach georglauterbach deleted the rspamd/4 branch January 30, 2023 07:23
@georglauterbach georglauterbach mentioned this pull request Feb 4, 2023
8 tasks
@georglauterbach georglauterbach added this to the v12.0.0 milestone Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants