https://github.com/spamhaus/spamassassin-dqs/blob/master/sh.cf#L7

spamhaus / spamassassin-dqs

Spamhaus code for the Spamassassin plugin. See https://docs.spamhaustech.com/40-real-world-usage/SpamAssassin/000-intro.html

Apache License 2.0

54 stars 16 forks source link

https://github.com/spamhaus/spamassassin-dqs/blob/master/sh.cf#L7 #7

Closed xpunkt closed 5 years ago

xpunkt commented 5 years ago

this line should not be in a cf file, and its part of spamassassin pre files so change or add it there :=)

ricalfieri commented 5 years ago

I put it there because we make use of shortcircuiting and maybe some people disabled/not enabled it in v320.pre file.

I guess I could add an "ifplugin Mail::SpamAssassin::Plugin::Shortcircuit" and then add some documentation to make people enable it from v320.pre , but I think it would be overkill.

I don't even see warnings from a "spamassassin -D --lint" when having it enabled both in sh.cf and v320.pre

ricalfieri commented 5 years ago

move the loading of shortcircuit in sh.pre