From: Matthew Vernon <matthewv@chiark.greenend.org.uk>
Date: Thu, 20 Mar 2014 12:26:42 +0000 (+0000)
Subject: Leave "Date" header alone.
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~webstump/git?a=commitdiff_plain;h=fea9ba5fbcb59686380c2b6c590a4b21b54af561;p=modbot-uram.git

Leave "Date" header alone.

Prior to this change, Stump removed the supplied Date: header, and
replaced it with one of its own, which reflected the time at which the
article was approved. Following discussion in unnm, it seemed clear
that people would like the Date: header to be unmolested.

Signed-off-by: Matthew Vernon <matthewv@chiark.greenend.org.uk>
---

diff --git a/stump/bin/processApproved b/stump/bin/processApproved
index a62333e..cd1f13c 100755
--- a/stump/bin/processApproved
+++ b/stump/bin/processApproved
@@ -40,14 +40,12 @@ post() {
     # I do grep above because a lot of users inserts empty
     # lines in the added headers.
 
-    echo Date: `date -R`
     cat $TMPFILE					\
       | formail -f -a "Newsgroups: $NEWSGROUP"		\
 	    -I Path:					\
             -I X-Moderate-For:                          \
             -I Return-Path:                             \
             -I X-Mailer:                                \
-	    -I "Date:"                                  \
             -I "X-400-Received:"                        \
 	    -I Received: -I "From " 			\
 	    -a "Approved: $PMUSER_APPROVAL"		\
diff --git a/stump/bin/submission.pl b/stump/bin/submission.pl
index e49680f..73c7239 100755
--- a/stump/bin/submission.pl
+++ b/stump/bin/submission.pl
@@ -270,7 +270,6 @@ sub ignoreHeader {
   return 1 if( $header =~ /^Supersedes:/i );
   return 1 if( $header =~ /^Precedence:/i );
   return 1 if( $header =~ /^Apparently-To:/i );
-  return 1 if( $header =~ /^Date:/i );
   return 1 if( $header =~ /^Expires:/i );
   return 1 if( $header =~ /^Distribution:/i );
   return 1 if( $header =~ /^Path:/i );