rspamd.com icon indicating copy to clipboard operation
rspamd.com copied to clipboard

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

Open styx-tdo opened this issue 1 year ago • 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.

styx-tdo avatar Apr 27 '24 19:04 styx-tdo