[rt-users] loop detection not working?

Phil Homewood pdh at snapgear.com
Tue Oct 1 21:46:44 EDT 2002


Dwayne Voelker wrote:
> it seems to me that rt should never send mail it thinks is a loop to
> requestor, is this right?

Yes, but there's an obscure bug in there somewhere.

A workaround exists as of 2.0.15pre1; if you're running 2.0.14 or
older, try the attched patch.
-- 
Phil Homewood, Systems Janitor, www.SnapGear.com
pdh at snapgear.com Ph: +61 7 3435 2810 Fx: +61 7 3891 3630
SnapGear - Custom Embedded Solutions and Security Appliances
-------------- next part --------------
Index: lib/RT/Interface/Email.pm
===================================================================
RCS file: /raid/cvsroot/rt/lib/RT/Interface/Email.pm,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- lib/RT/Interface/Email.pm	3 May 2002 05:30:23 -0000	1.5
+++ lib/RT/Interface/Email.pm	16 Aug 2002 20:09:12 -0000	1.6
@@ -1,4 +1,4 @@
-# $Header: /raid/cvsroot/rt/lib/RT/Interface/Email.pm,v 1.5 2002/05/03 05:30:23 jesse Exp $
+# $Header: /raid/cvsroot/rt/lib/RT/Interface/Email.pm,v 1.6 2002/08/16 20:09:12 jesse Exp $
 # RT is (c) 1996-2001 Jesse Vincent <jesse at fsck.com>
 
 package RT::Interface::Email;
@@ -12,7 +12,7 @@
     use vars qw ($VERSION  @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
     
     # set the version for version checking
-    $VERSION = do { my @r = (q$Revision: 1.5 $ =~ /\d+/g); sprintf "%d."."%02d" x $#r, @r }; # must be all one line, for MakeMaker
+    $VERSION = do { my @r = (q$Revision: 1.6 $ =~ /\d+/g); sprintf "%d."."%02d" x $#r, @r }; # must be all one line, for MakeMaker
     
     @ISA         = qw(Exporter);
     
@@ -152,6 +152,7 @@
     #If this instance of RT sent it our, we don't want to take it in
     my $RTLoop = $head->get("X-RT-Loop-Prevention") || "";
     chomp ($RTLoop); #remove that newline
+    $RTLoop =~ s/\s*$//;
     if ($RTLoop eq "$RT::rtname") {
 	return (1);
     }


More information about the rt-users mailing list