6.0.0-beta1
7/7/25

[#13832] Mailer - send FQDN with the HELO command
Summary Mailer - send FQDN with the HELO command
Queue Horde Framework Packages
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester guillaume (at) majbe (dot) net
Created 01/29/2015 (3812 days ago)
Due
Updated 01/31/2015 (3810 days ago)
Assigned 01/29/2015 (3812 days ago)
Resolved 01/31/2015 (3810 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
01/31/2015 11:25:57 PM Michael Slusarz Comment #4
State ⇒ Resolved
Reply to this comment
Horde_Smtp 1.9.0
01/31/2015 11:20:46 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit d543359bf737093f514bd6adf9aa2bfae2475d85
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Sat Jan 31 16:13:48 2015 -0700

     [mms] Add 'localhost' parameter to Horde_Smtp (Request #13832).

  framework/Smtp/lib/Horde/Smtp.php |   20 ++++++++++++++++++--
  framework/Smtp/package.xml        |   12 ++++++------
  2 files changed, 24 insertions(+), 8 deletions(-)

http://github.com/horde/horde/commit/d543359bf737093f514bd6adf9aa2bfae2475d85
01/29/2015 01:35:16 PM Jan Schneider Comment #2
State ⇒ Assigned
Priority ⇒ 1. Low
Assigned to Michael Slusarz
Reply to this comment
Looks fine to me.
01/29/2015 09:58:29 AM guillaume (at) majbe (dot) net Comment #1
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Mailer - send FQDN with the HELO command
Queue ⇒ Horde Framework Packages
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
The problem :
-----------------

sending e-mail with IMP returned an error because the SMTP Mailer 
didn't give the machine's FQDN in the HELO dialog - our MSA is a bit 
strict about that.

We properly positionned the /$conf['mailer']['params']['localhost']/ 
option, but didn't get any positive feedback.

Investigations told that the file /usr/share/php/Horde/Smtp.php ( 
Debian Wheezy ) stated :

  824     protected function _hello()
  825     {
  826         $ehlo = $host = gethostname();
  827         if ($host === false) {
  828             $ehlo = $_SERVER['SERVER_ADDR'];
  839             $host = 'localhost';
  830         }

A solution :
--------------

Line 826 : $ehlo = $host = $this->getParam('localhost');

I don't know if it is The Right Way To Do, but it does the job :)

I'd be thankful if you may correct it upstream.

Sincerely yours,
Guillaume.

Saved Queries