6.0.0-beta1
7/8/25

[#14408] addReceivedHeader() interaction with SpamAssassin trustedpath
Summary addReceivedHeader() interaction with SpamAssassin trustedpath
Queue Horde Framework Packages
Type Bug
State Not A Bug
Priority 2. Medium
Owners mrubinsk (at) horde (dot) org
Requester admin (at) dei (dot) unipd (dot) it
Created 06/22/2016 (3303 days ago)
Due
Updated 06/24/2016 (3301 days ago)
Assigned 06/22/2016 (3303 days ago)
Resolved 06/24/2016 (3301 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
06/24/2016 09:25:41 AM Jan Schneider Comment #4
State ⇒ Not A Bug
Reply to this comment
So it's up to SpamAssassin to update their rules.
06/23/2016 10:02:54 PM Michael Rubinsky Comment #3 Reply to this comment
Michael, you introduced this, can you explain the rationale?
Yes, it provides a more complete and accurate audit trail. This helps 
when the email in question must be shown to be transmitted in a secure 
manner. For example, email that contains protected health information 
covered by HIPAA must have an audit trail that shows completely secure 
transmission (a minimum of TLS security and if webmail is used, SSL is 
reqiured).
06/22/2016 02:19:23 PM Jan Schneider Comment #2
Assigned to Michael Rubinsky
State ⇒ Feedback
Reply to this comment
Michael, you introduced this, can you explain the rationale?
06/22/2016 01:23:21 PM admin (at) dei (dot) unipd (dot) it Comment #1
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ addReceivedHeader() interaction with SpamAssassin trustedpath
Queue ⇒ Horde Framework Packages
Milestone ⇒
Patch ⇒ Yes
State ⇒ Unconfirmed
Reply to this comment
Horde_Pear (2.9.5)
Horde/Mime/Headers/Deprecated.php
function addReceivedHeader() adds:
Received from ... by ... (Horde Framework) with HTTP; ...

if $_SERVER['HTTPS'] is "on" adds:
Received from ... by ... (Horde Framework) with HTTPS; ...

This is "correct" but Spamassassin doesnt recognize it as a 
authenticated user on trusted path
and check message against all remote IP rules.

Q&D workaround:
diff /usr/share/pear/Horde/Mime/Headers/Deprecated.php.orig 
/usr/share/pear/Horde/Mime/Headers/Deprecated.php
146c146
<             ($is_ssl ? 'S' : '') . '; ' . date('r')
---
             ($is_ssl ? '' : '') . '; ' . date('r')
I will open a similar bug on Spamassassin:
diff 
/usr/share/perl5/vendor_perl/Mail/SpamAssassin/Message/Metadata/Received.pm*
408c408
<   if (/ by / && / with ((?:ES|L|UTF8S|UTF8L)MTPS?A|ASMTP|HTTPU?)(?: 
|;|$)/i) {
---
   if (/ by / && / with 
((?:ES|L|UTF8S|UTF8L)MTPS?A|ASMTP|HTTP(S|U)?)(?: |;|$)/i) {
Regards
Valerio Pulese

Saved Queries