Mailing List Archive

And it is even mentioned in the UPGRADE notes:

- The HashBL plugin in 342.pre is now enabled by default.

(sad typo in the filename)


On Wed, Dec 28, 2022 at 04:21:45PM +0200, Henrik K wrote:
>
> This was discussed and approved in some of the 4.0.0 bugs. There should be
> no need to revisit it. It still wouldn't make sense to have loadplugin
> HashBL in two *.pre files.
>
> On Wed, Dec 28, 2022 at 09:18:51AM -0500, Kevin A. McGrail wrote:
> > Wow, as it's enabled in v342.pre, that would imply it was enabled in 3.4.2.?
> > We should not have changed a past pre file for the 4.0.0 release IMO but
> > added it to the 4.0.0.pre file.? Such is life.? Should we fix it for 4.0.1?
> >
> > On 12/28/2022 9:07 AM, Henrik K wrote:
> > > Just keep in mind that HashBL is only enabled for fresh 4.0.0 installs, it
> > > wasn't default previously.
> >
> > --
> > Kevin A. McGrail
> > KMcGrail@Apache.org
> >
> > Member, Apache Software Foundation
> > Chair Emeritus Apache SpamAssassin Project
> > https://www.linkedin.com/in/kmcgrail - 703.798.0171

On 28/12/22 15:15, Henrik K wrote:

>
> Maybe would be even good idea to use something like this:
>
> ifplugin Mail::SpamAssassin::Plugin::HashBL
> ....
> else
> error: Please activate HashBL plugin in v342.pre
> endif
I think I'll just add the ifplugin condition in the two .cf files and
add a note in the README. No reason to overengineer something that it
should be working by default, as it is in a stock SA installation.

--
Best regards,
Riccardo Alfieri

Spamhaus Technology
https://www.spamhaus.com/

Riccardo Alfieri skrev den 2022-12-28 11:44:
> Hello everyone,
>
> just FYI, I published the updated rules to have DQS working on SA
> 4.0.0+ (https://github.com/spamhaus/spamassassin-dqs)

https://github.com/spamhaus/spamassassin-dqs/blob/master/4.0.0%2B/sh.cf

dated Spamhaus's SpamAssassin setup version 20220420

imho not the sa 4.x.x version ?

>
> Thanks to the effort of all SA developers there is no need anymore to
> install a dedicated plugin, as all of our functions have been
> backported in SA's core code, so now it's only a bunch of .cf to copy.
>
> Please consider the rules as a BETA. They have been tested by a few
> customers without issues, but YMMV.