Benny Pedersen wrote on 4/01/23 11:00 am:
https://github.com/apache/spamassassin/blob/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm#L289
$sql .= " ON CONFLICT (username, email, signedby, ip) DO UPDATE set
msgcount = ?, totscore += ?";
confirm is from my side needed it would fix it, i atleast like to think
it does
There's a typo, which must just be in your email since postgres won't
accept it, that should be =+ not +=
there is more lines with totscore = totscore that might need fix aswell
I checked that, and the only two other places I saw were conditional on
not being postgres. Also, the last place it is used, in the fallback
UPDATE statement, I tried it in postgres and found that in that
statement postgres allows the totscore = totscore + 10 syntax. Strange.
I opened a bug https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8098
and will submit a patch for that =+ change.
Thanks for catching this.
Sidney
