6.0.0-beta6
3/30/26
  • Error
    Ticket 65576 was not found.

[#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 1/29/15 (4078 days ago)
Due
Updated 1/31/15 (4076 days ago)
Assigned 1/29/15 (4078 days ago)
Resolved 1/31/15 (4076 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