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 2020/07/27 08:53:16 UTC

svn commit: r1880351 - /spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm

Author: gbechis
Date: Mon Jul 27 08:53:16 2020
New Revision: 1880351

URL: http://svn.apache.org/viewvc?rev=1880351&view=rev
Log:
remove 3.4 compat code from trunk version of the plugin

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm?rev=1880351&r1=1880350&r2=1880351&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/FromNameSpoof.pm Mon Jul 27 08:53:16 2020
@@ -113,7 +113,6 @@ use warnings;
 use re 'taint';
 
 use Mail::SpamAssassin::Plugin;
-use Mail::SpamAssassin::Util;
 
 use vars qw(@ISA);
 @ISA = qw(Mail::SpamAssassin::Plugin);
@@ -122,18 +121,6 @@ my $VERSION = 0.9;
 
 sub dbg { Mail::SpamAssassin::Plugin::dbg ("FromNameSpoof: @_"); }
 
-sub uri_to_domain {
-  my ($self, $domain) = @_;
-
-  return unless defined $domain;
-
-  if ($Mail::SpamAssassin::VERSION <= 3.004000) {
-    Mail::SpamAssassin::Util::uri_to_domain($domain);
-  } else {
-    $self->{main}->{registryboundaries}->uri_to_domain($domain);
-  }
-}
-
 # constructor: register the eval rule
 sub new
 {
@@ -369,9 +356,9 @@ sub _check_fromnamespoof
 
   $tod{'addr'} = lc $toaddrs[0];
 
-  $fnd{'domain'} = $self->uri_to_domain($fnd{'addr'});
-  $fad{'domain'} = $self->uri_to_domain($fad{'addr'});
-  $tod{'domain'} = $self->uri_to_domain($tod{'addr'});
+  $fnd{'domain'} = $self->{main}->{registryboundaries}->uri_to_domain($fnd{'addr'});
+  $fad{'domain'} = $self->{main}->{registryboundaries}->uri_to_domain($fad{'addr'});
+  $tod{'domain'} = $self->{main}->{registryboundaries}->uri_to_domain($tod{'addr'});
 
   return 0 unless (defined $fnd{'domain'} && defined $fad{'domain'});
 
@@ -421,7 +408,7 @@ sub _find_address_owner
     }
   }
 
-  my $owner = $self->uri_to_domain($check);
+  my $owner = $self->{main}->{registryboundaries}->uri_to_domain($check);
 
   $check =~ /^([^\@]+)\@(.*)$/;