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 a62333ecd60c78160bccb07f5c239c2ebe745ee3..cd1f13c32e2cd19bb274a19c3bc79c07d8d5905e 100755 (executable)
--- 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 e49680f66b8b3ac28fa45f7b22b473a0b63de980..73c72398696eb35a83bc9cf039e1d52209c9ff66 100755 (executable)
--- 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 );