[imp] [patch] Configurable Received: header behavior

[Thomas Bolioli]

Thanks for this. It is probably best to provide the option to the end 
user but I would leave the default observing the RFC specs and allow 
override from that.
Tom

Brian Keifer wrote:

>Hi, All.
>
>I've been seeing some people mentioning that they have issues with spam filters
>seeing their dial-up or otherwise dynamic IP in the Received headers and
>thinking the messages are originating from a mail server in dynamic space.
>I've been bitten by this as well recently, so I wanted to do something about
>it, if only for my own server.
>
>The attached file is a diff against imp-h3-4.0.2-rc1.  It adds an admin-level
>config option (on by default) to enable or disable adding the first received
>header.  I wanted to toss this out as a suggestion, mainly to see if the core
>folks had any strong feelings either way on it.
>
>I've checked out a couple of webmail services (Yahoo!, Hotmail, Gmail, and a
>couple of local ISPs) and so far, Gmail's the only one that doesn't include the
>sending workstation's IP address in a Received: header.
>
>If it's dubbed fit for inclusion, I'll be happy to re-roll a patch against HEAD,
>but as you can see, it's a fairly simple one. =)
>
>-Brian
>
>  
>
>------------------------------------------------------------------------
>
>diff -ur imp-h3-4.0.2-rc1/compose.php imp/compose.php
>--- imp-h3-4.0.2-rc1/compose.php	Tue Feb  1 01:40:50 2005
>+++ imp/compose.php	Wed Feb  9 09:45:51 2005
>@@ -841,8 +841,12 @@
>     /* Initalize a header object for the outgoing message. */
>     $msg_headers = &new IMP_Headers();
> 
>-    /* Add a Received header for the hop from browser to server. */
>-    $msg_headers->addReceivedHeader();
>+    /* Add a Received header for the hop from browser to server*/
>+    if ($conf['msg']['received_for_browser']) {
>+        $msg_headers->addReceivedHeader();
>+    }
>+
>+    /* Add a MessageID header */
>     $msg_headers->addMessageIdHeader();
> 
>     /* Add the X-Priority header, if requested. This appears here since
>diff -ur imp-h3-4.0.2-rc1/config/conf.xml imp/config/conf.xml
>--- imp-h3-4.0.2-rc1/config/conf.xml	Thu Dec  9 10:33:27 2004
>+++ imp/config/conf.xml	Wed Feb  9 09:43:36 2005
>@@ -197,6 +197,8 @@
>    <configboolean name="append_trailer" desc="Should we append the contents of
>    imp/config/trailer.txt to the end of every message
>    sent?">true</configboolean>
>+   <configboolean name="received_for_browser" desc="Should we add a Received: 
>+   header for the hop between the browser and the server?">true</configboolean>
>   </configsection>
> 
>   <configsection name="compose">
>  
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3194 bytes
Desc: S/MIME Cryptographic Signature
Url : http://lists.horde.org/archives/imp/attachments/20050209/826c497d/smime.bin