Admin copy of error email from mailgw includes traceback

Author: Richard Jones

CHANGES.txt

@@ -1,13 +1,19 @@
 This file contains the changes to the Roundup system over time. The entries
 are given with the most recent entry first.
 
-2007-11-?? 1.4.1
-Fixed:
-- Removed some metakit references
+2007-??-?? 1.4.2
+Feature:
 - New config option in mail section: ignore_alternatives allows to
   ignore alternatives besides the text/plain part used for the content
   of a message in multipart/alternative attachments.
 - Handle no time.tzset under Windows (sf #1825643)
+- Admin copy of error email from mailgw includes traceback (thanks Ulrik
+  Mikaelsson)
+
+
+2007-11-09 1.4.1
+Fixed:
+- Removed some metakit references
 
 
 2007-11-04 1.4.0

doc/index.txt

@@ -129,6 +129,7 @@ Will Maier,
 Georges Martin,
 Gordon McMillan,
 John F Meinel Jr,
+Ulrik Mikaelsson,
 John Mitchell,
 Ramiro Morales,
 Toni Mueller,

roundup/mailgw.py

@@ -73,7 +73,7 @@ class node. Any parts of other types are each stored in separate files
 an exception, the original message is bounced back to the sender with the
 explanatory message given in the exception.
 
-$Id: mailgw.py,v 1.193 2007-11-14 14:57:47 schlatterbeck Exp $
+$Id: mailgw.py,v 1.194 2007-11-19 20:45:57 richard Exp $
 """
 __docformat__ = 'restructuredtext'
 
@@ -764,12 +764,15 @@ def handle_Message(self, message):
 
             # bounce the message back to the sender with the error message
             # let the admin know that something very bad is happening
-            sendto = [sendto[0][1], self.instance.config.ADMIN_EMAIL]
             m = ['']
             m.append('An unexpected error occurred during the processing')
             m.append('of your message. The tracker administrator is being')
             m.append('notified.\n')
-            self.mailer.bounce_message(message, sendto, m)
+            self.mailer.bounce_message(message, sendto[0][1], m)
+
+            m.append('----------------')
+            m.append(traceback.format_exc())
+            self.mailer.bounce_message(message, [self.instance.config.ADMIN_EMAIL], m)
 
     def handle_message(self, message):
         ''' message - a Message instance