Tickets :: [#14574] Invalid Received header

6.0.0-beta1

7/3/25

Summary	Invalid Received header
Queue	Horde Framework Packages
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	azurit (at) pobox (dot) sk
Created	02/06/2017 (3069 days ago)
Due
Updated	02/11/2017 (3064 days ago)
Assigned
Resolved	02/07/2017 (3068 days ago)
Github Issue Link
Github Pull Request
Milestone	Horde_Core 2.27.7
Patch	No

02/11/2017 12:27:02 PM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (master): commit 4caf0e71a91b551084b7ba296979b57e43d3c709 Author: Jan Schneider <jan@horde.org> Date: Sat Feb 11 13:26:46 2017 +0100 [jan] Correctly encode IP address literal in first Received: header (~~Bug #14574~~). framework/Mime/package.xml \| 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) http://github.com/horde/horde/commit/4caf0e71a91b551084b7ba296979b57e43d3c709

02/10/2017 08:48:28 PM	azurit (at) pobox (dot) sk	Comment #6	Reply to this comment
looks good, thank you!

02/10/2017 08:37:17 PM	Michael Rubinsky	Comment #5	Reply to this comment
How about now?

02/10/2017 08:36:15 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (master): commit 5ace791ba4f4a29e76d417a216faae537623d20c Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Fri Feb 10 15:34:54 2017 -0500 ~~Bug: 14574~~ Patch the deprecated method, too, for FW_52 installs. framework/Mime/lib/Horde/Mime/Headers/Deprecated.php \| 4 ++++ 1 file changed, 4 insertions(+) http://github.com/horde/horde/commit/5ace791ba4f4a29e76d417a216faae537623d20c

02/07/2017 12:09:25 PM	azurit (at) pobox (dot) sk	Comment #3	Reply to this comment
I see some issues with the patch. First, file Horde/Core/Mime/Headers/Received.php is never used in my installation, the right one is Horde/Mime/Headers/Deprecated.php . Second, this header was generated: Received: from [] ([1.2.3.4]) by webmail.example.com (Horde Framework) with HTTPS; Tue, 07 Feb 2017 13:06:09 +0100

02/07/2017 10:57:16 AM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ Horde_Core 2.27.7

02/07/2017 10:56:58 AM	Jan Schneider	Version ⇒ Queue ⇒ Horde Framework Packages

02/07/2017 10:56:18 AM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git (master): commit 1247789eea27cafebdbb2a8ef086cd0d7b87b2a0 Author: Jan Schneider <jan@horde.org> Date: Tue Feb 7 11:56:00 2017 +0100 [jan] Correctly encode IP address literal in first Received: header (~~Bug #14574~~). framework/Core/lib/Horde/Core/Mime/Headers/Received.php \| 7 ++++++- framework/Core/package.xml \| 2 ++ 2 files changed, 8 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/1247789eea27cafebdbb2a8ef086cd0d7b87b2a0

02/06/2017 07:00:42 PM	azurit (at) pobox (dot) sk	Comment #1 Milestone ⇒ State ⇒ Unconfirmed Patch ⇒ No Queue ⇒ IMP Summary ⇒ Invalid Received header Type ⇒ Bug Priority ⇒ 2. Medium	Reply to this comment
IMP is generating this Received header which contains IP address of a client: Received: from 1.2.3.4 ([1.2.3.4]) by webmail.example.com (Horde Framework) with HTTPS; Mon, 06 Feb 2017 13:00:50 +0100 The problem is that the first IP address (within no brackets) is supposed to be an HELO/EHLO string. If HELO/EHLO is an IP address, it must be inside "[" and "]" brackets. The headers should look like this: Received: from [1.2.3.4] ([1.2.3.4]) by webmail.example.com (Horde Framework) with HTTPS; Mon, 06 Feb 2017 13:00:50 +0100 Current format is doing problems for example with SpamAssassin: E-mails are fireing RCVD_NUMERIC_HELO and FSL_HELO_BARE_IP_2 spam checks. You can try it here: http://spamcheck.postmarkapp.com/