Add a "Date:" header to the mail

[grg]

Hello,

I've installed trac-wiki-notify and one user reported that his mail client shows the date as follows:

Date: 12/31/69

This is because trac-wiki-notify does not add any Date: header, and some mail clients show a weird default date instead.

I've modified trac-wiki-notify to add the Date: header and would like to contribute this feature for further inclusion into the official version. Here is the patch:

trac-wiki-notify

@@ -35 +35 @@
 import sys
 import smtplib
 import re
+
+# line added by grg@openwall.com
+from time import tzname, timezone, daylight, altzone
+
 from time import strftime, localtime, time
 
+# function added by grg@openwall.com
+def grg_tz_offset():
+  lt = localtime()
+  if lt.tm_isdst > 0 and daylight:
+      tz = tzname[1]
+      utc_offset_minutes = - int(altzone/60)
+  else:
+      tz = tzname[0]
+      utc_offset_minutes = - int(timezone/60)
+  utc_offset_str = "%+03d%02d" % (utc_offset_minutes/60.0, utc_offset_minutes % 60)
+  return utc_offset_str
+
+# function added by grg@openwall.com
+def grg_timestamp():
+  return strftime('%a, %d %b %Y %H:%M:%S ' + grg_tz_offset())
+
 if len(sys.argv) < 5:
     raise StandardError("Not enough arguments")
 
@@ -82 +102 @@
         for email in smtp_to:
             smtp_body = "From: %s\n" % smtp_from
             smtp_body += "To: %s\n" % email
+            # line added by grg@openwall.com
+            smtp_body += "Date: %s\n" % grg_timestamp()
             smtp_body += "Reply-To: %s\n" % smtp_from
             smtp_body += "Subject: Notification of %i changes to Trac Wiki %s\n\n" % (len(changes), trac_url)
             smtp_body += email_body

[grg]

The patch is not correctly displayed by the wiki, please see the file trac-wiki-notify.header.date.diff attached to this ticket.

[Noah Kantrowitz]

Fixing formatting.

[Ryan J Ollos]

This script is deprecated. Please take a look at the AnnouncerPlugin and WatchlistPlugin.