Skip to content

mod_spam_filter: handle rtbl blocked domains#353

Merged
weiss merged 3 commits intoprocessone:masterfrom
sstrigler:mod_spam_filter-rtbl-integration
Apr 4, 2025
Merged

mod_spam_filter: handle rtbl blocked domains#353
weiss merged 3 commits intoprocessone:masterfrom
sstrigler:mod_spam_filter-rtbl-integration

Conversation

@sstrigler
Copy link
Contributor

This adds support for RTBL blocked domains similar to how prosody's mod_anti_spam does it, following the guide at https://xmppbl.org/dev. It subscribes to the spam_source_domains pubsub node at the given rtbl_host (eg. use "xmppbl.org") and retrieves a list of domains. Presence and message stanzas from there are treated like block-strangers, means only if there's a roster subscription they are being let through.

I've add the following commands:

  • get_blocked_domains
  • add_blocked_domain
  • add_to_spam_filter_cache

@sstrigler sstrigler marked this pull request as draft April 1, 2025 14:26
@sstrigler sstrigler force-pushed the mod_spam_filter-rtbl-integration branch from e143f2e to 01dfa3a Compare April 1, 2025 15:02
@sstrigler sstrigler marked this pull request as ready for review April 1, 2025 15:03
@sstrigler sstrigler force-pushed the mod_spam_filter-rtbl-integration branch from b0df451 to 27d9efa Compare April 2, 2025 08:57
@badlop
Copy link
Member

badlop commented Apr 2, 2025

Now that the PR passes the tests, it can be merged whenever you consider it stable enough.

Otherwise, if you still want to make improvements, can you please rebase this branch with recent "master"?

Specifically to get commit 3c10f86: the test action in this repository pulled elixir container images from docker hub, and it seems docker hub now has a pull quota... so I've updated to another container registry.

@sstrigler
Copy link
Contributor Author

Please merge it then. I’ll create extra PRs for upcoming changes.

@weiss weiss merged commit 1bfe7a6 into processone:master Apr 4, 2025
6 checks passed
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