You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by gb...@apache.org on 2023/05/05 07:19:33 UTC
svn commit: r1909622 - in /spamassassin/trunk/lib/Mail/SpamAssassin: Plugin/TxRep.pm SQLBasedAddrList.pm
Author: gbechis
Date: Fri May 5 07:19:33 2023
New Revision: 1909622
URL: http://svn.apache.org/viewvc?rev=1909622&view=rev
Log:
better regexp, spotted by hege@, thanks
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TxRep.pm
spamassassin/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TxRep.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TxRep.pm?rev=1909622&r1=1909621&r2=1909622&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TxRep.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/TxRep.pm Fri May 5 07:19:33 2023
@@ -1831,7 +1831,7 @@ sub pack_addr {
if ( $self->{conf}->{txrep_welcomelist_out} &&
defined $self->{pms}->{relays_internal} && @{$self->{pms}->{relays_internal}} &&
(!defined $self->{pms}->{relays_external} || !@{$self->{pms}->{relays_external}})
- and $addr =~ /\@\w+\./) {
+ and $addr =~ /(?:[^\s\@]+)\@(?:[^\s\@]+)/) {
$origip = 'WELCOMELIST_OUT';
}
return $addr . "|ip=" . $origip;
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm?rev=1909622&r1=1909621&r2=1909622&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/SQLBasedAddrList.pm Fri May 5 07:19:33 2023
@@ -204,7 +204,7 @@ sub get_addr_entry {
my $sql;
my $sth;
my $rc;
- if($self->{main}->{conf}->{txrep_welcomelist_out} and ($email =~ /\@\w+\./)) {
+ if($self->{main}->{conf}->{txrep_welcomelist_out} and ($email =~ /(?:[^\s\@]+)\@(?:[^\s\@]+)/)) {
$sql = "SELECT msgcount, totscore FROM $self->{tablename} " .
"WHERE username = ? AND email = ? AND ip = 'WELCOMELIST_OUT'";
$sth = $self->{dbh}->prepare($sql);
@@ -329,7 +329,7 @@ sub add_score {
{ my @fields = qw(username email ip msgcount totscore);
my @signedby;
- if ($self->{_with_awl_signer} or (defined $signedby and $signedby =~ /^spf\-/) and not ($self->{main}->{conf}->{txrep_welcomelist_out} and ($email =~ /\@\w+\./)) ) {
+ if ($self->{_with_awl_signer} or (defined $signedby and $signedby =~ /^spf\-/) and not ($self->{main}->{conf}->{txrep_welcomelist_out} and ($email =~ /(?:[^\s\@]+)\@(?:[^\s\@]+)/)) ) {
push(@fields, 'signedby');
@signedby = !defined $signedby ? () : split(' ', lc $signedby);
@signedby = ( '' ) if !@signedby;