rspamd/rspamd.com

reputation - main config, docs and local; redis/sentinel not taken from global

Opened this issue · 0 comments

In the modules.d/reputation.conf:, it gives:

reputation {
#  rules { 
#    SPF_REPUTATION =  {
#      selector {
#        type = "spf";
#      }
#      backend {
#        type = "redis";
#      }
#      symbol = "SPF_REPUTATION";
#    }
#  }
  .include(try=true,priority=5) "${DBDIR}/dynamic/reputation.conf"
  .include(try=true,priority=1,duplicate=merge) "$LOCAL_CONFDIR/local.d/reputation.conf"
  .include(try=true,priority=10) "$LOCAL_CONFDIR/override.d/reputation.conf"
}   

The commented syntax should be what one can put in local config, however, the local syntax is:
https://rspamd.com/doc/modules/reputation.html

rules {
  ip_reputation = {
    selector "ip" {
    }
    backend "redis" {
      servers = "localhost";
    }
    symbol = "IP_REPUTATION";
  }

This is: a) confusing and b) does not allow to set the backend to redis and use the redis.conf settings (leaving servers empty fails configtest, putting sentinels in there also fails.

Please fix the reputation module.