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.

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

Add cmdline option for rspamd Settings-ID HTTP header #36

Merged
merged 2 commits into from
Nov 30, 2020
Merged

Add cmdline option for rspamd Settings-ID HTTP header #36

merged 2 commits into from
Nov 30, 2020

Conversation

mneumann
Copy link
Contributor

This allows to distinguish between multiple rspamd settings
by using separate opensmtpd filters with different -settings-id parameters.

For example:

dkim {
	id = "dkim";
	apply {
		groups_enabled = ["dkim"];
	}
}
filter rspamd-dkim proc-exec "opensmtpd-filter-rspamd -url http://localhost:11333 -settings-id dkim"

This allows to distinguish between multiple rspamd settings
by using separate opensmtpd filters with different -settings-id parameters.

For example:

```
dkim {
	id = "dkim";
	apply {
		enable_groups = ["dkim"];
	}
}
```

```
filter rspamd-dkim proc-exec "opensmtpd-filter-rspamd -url http://localhost:11333 -settings-id dkim"
```
@mikejsavage
Copy link

I ran into this a few days ago, so specifically the issue is that you want outgoing mail to go through rspamd for dkim signing but not any spam checking, because sending mails with X-Spam: yes headers is awkward

I resolved this by using filter-dkimsign (pkg_add opensmtpd-filter-dkimsign on openbsd)

@mneumann
Copy link
Contributor Author

mneumann commented Sep 8, 2020

@mikejsavage exactly! Having X-Spam: yes headers in your outgoing email is ugly. And for some reason, any outgoing mail gets classified as Spam on my system (probably a bug/misconfiguration around rspamd). I will try to look at filter-dkimsign, thanks!

@poolpOrg
Copy link
Owner

I missed this PR somehow, can you please update the README.md to document the new option ?

@mneumann
Copy link
Contributor Author

@poolpOrg Added a short comment in the README.md on how to use it. Please take a look if that makes sense.

@poolpOrg
Copy link
Owner

Perfect

@poolpOrg poolpOrg merged commit 0f0aeab into poolpOrg:master Nov 30, 2020
@mneumann mneumann deleted the feature/add-settings-id branch November 30, 2020 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants