https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6416
Justin Mason <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #2 from Justin Mason <[email protected]> 2010-04-21 11:47:25 EDT --- Mark: @@ -336,12 +338,13 @@ $addr = lc $addr; $addr =~ s/[\000\;\'\"\!\|]/_/gs; # paranoia + if (defined $origip) { + $origip = $self->ip_to_awl_key($origip); + } if (!defined $origip) { # could not find an IP address to use, could be localhost mail # or from the user running "add-addr-to-*". $origip = 'none'; - } else { - $origip = $self->ip_to_awl_key($origip); } return $addr . "|ip=" . $origip; } I don't understand the reasoning here. why does this move the else clause? -- Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.
