Developers of Sympa

[David Verdin <address@concealed>]

So, finally:

This problem is fixed in the stable (5.3) branch. Please find a patch 
attached.
This problem treatment is improved in the development branch, and then 
in the future 5.4, with the introduction of a "bad" directory for 
bounces (and archives, by the way). Any bounce we can't handle will be 
stored in a "bad" directory. If the listmaster can do something, (s)he 
will be notified.

Regards,

Micah Anderson a écrit :
> Hi,
>
> First of all, I tried to report this as a bug in the tracker, but I was 
> unable to do so. When I click Login, I am directed to the https://
> federation.cru.fr website and from there I am able to login, but I cannot 
> return to the bug reporting page where I can report a bug while logged in.
>
> The problem I am experiencing is that since I've upgraded to 5.3.4 
> bounced.pl keeps dying mysteriously. I decided I needed to run it in 
> debug mode to determine what is causing the crash, however running with 
> the -d switch eventually results in bounced.pl dying because the function 
> DoForward() does not exist in bounced.pl. It seems as if this function 
> exists in sympa.pl but without it available in bounced.pl its not 
> possible to run bounced.pl in debug mode:
>
> List::init_list_cache() List::init_list_cache()
> bounce for :address@concealed:  Conf
> {bounce_email_prefix}=bouncexx
> processing  Email Feedback Report
> ignoring Feedback Report address@concealed : unknown 
> format (feedback_type:abuse, original_rcpt:, 
> listname:address@concealed
> )
> Undefined subroutine &main::DoForward called at ./bounced.pl line 376.
>
> If I copy the DoForward subroutine into bounced.pl then I can run in 
> debug mode. I presume that the subroutines located in sympa.pl are 
> normally available to bounced.pl?
>
> micah
>   

--
David Verdin
Comité réseau des universités

Index: wwsympa/bounced.pl
===================================================================
--- wwsympa/bounced.pl	(revision 4721)
+++ wwsympa/bounced.pl	(working copy)
@@ -369,11 +369,9 @@
 		}
 	    
 	    }else{
-		$forward = 'listmaster';
 		do_log ('err','ignoring Feedback Report %s : unknown format (feedback_type:%s, original_rcpt:%s, listname:%s)',$file, $feedback_type, $original_rcpt, $listname );		
 	    }
-	    $listname ||= 'sympa';
-	    &DoForward($listname, $forward, $robot, $entity) if (defined $forward);
+	    &List::send_notify_to_listmaster('intern_error',$Conf{'domain'},{'error' => "Ignoring Feedback Report %s : unknown format (feedback_type:$feedback_type, original_rcpt:$original_rcpt, listname:$listname)"});
 
 	    unlink("$queue/$file");
 	    next;		
Index: mail_tt2/listmaster_notification.tt2
===================================================================
--- mail_tt2/listmaster_notification.tt2	(revision 4721)
+++ mail_tt2/listmaster_notification.tt2	(working copy)
@@ -217,6 +217,14 @@
 
 [%|loc%]Managing bouncing subscribers:[%END%] [% conf.wwsympa_url %]/reviewbouncing/[% list.name %]
 
+[% ELSIF type == 'intern_error' -%]
+Subject: [% FILTER qencode %][%|loc(list.name)%]Listmaster : internal server error [%END%][%END%]
+
+[%IF error%][%error%]
+[%END%]  
+
+[%|loc%]See logs for more details.[%END%]
+
 [%# INTERNAL SERVER ERROR  : MAIL -%]
 [% ELSIF type == 'mail_intern_error' -%]
 Subject: [% FILTER qencode %][%|loc(list.name)%]Listmaster : internal server error [%END%][%END%]