- Added logging for template errors in HyperArch.py. (LP: #558254)

2 files changed, 4 insertions, 2 deletions

diff --git a/Mailman/Archiver/HyperArch.py b/Mailman/Archiver/HyperArch.py
index 33a77f0b..a7f46276 100644
--- a/Mailman/Archiver/HyperArch.py
+++ b/Mailman/Archiver/HyperArch.py
@@ -226,9 +226,9 @@ def quick_maketext(templatefile, dict=None, lang=None, mlist=None):
                                     Utils.GetCharSet(lang),
                                     'replace')
                 text = sdict.interpolate(utemplate)
-        except (TypeError, ValueError):
+        except (TypeError, ValueError), e:
             # The template is really screwed up
-            pass
+            syslog('error', 'broken template: %s\n%s', filepath, e)
     # Make sure the text is in the given character set, or html-ify any bogus
     # characters.
     return Utils.uncanonstr(text, lang)
diff --git a/NEWS b/NEWS
index 622fca81..69f47e4f 100755
--- a/NEWS
+++ b/NEWS
@@ -61,6 +61,8 @@ Here is a history of user visible changes to Mailman.
 
   Bug Fixes and other patches
 
+    - Added logging for template errors in HyperArch.py.  (LP: #558254)
+
     - Added more explanation to the bad owner address message from
       bin/newlist.  (LP: #1200763)