[mms] Fix handling servers that don't support ESMTP (Bug #12853).

horde · Nov 21, 2013 · 0f8aa28 · 0f8aa28
1 parent fb255f0
commit 0f8aa28
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/framework/Smtp/lib/Horde/Smtp.php b/framework/Smtp/lib/Horde/Smtp.php
@@ -379,6 +379,12 @@ public function login()
             case 502:
                 // Old server - doesn't support EHLO
                 $this->_connection->write('HELO ' . $host);
+                try {
+                    $this->_getResponse(250);
+                } catch (Horde_Smtp_Exception $e2) {
+                    $this->logout();
+                    throw $e;
+                }
                 $this->_extensions = array();
                 break;
 

diff --git a/framework/Smtp/package.xml b/framework/Smtp/package.xml
@@ -21,6 +21,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
  <notes>
+* [mms] Fix handling servers that don&apos;t support ESMTP (Bug #12853).
 * [mms] Fix returning reply code in Exceptions.
  </notes>
  <contents>
@@ -304,6 +305,7 @@
    <date>2013-10-30</date>
    <license uri="http://www.horde.org/licenses/lgpl21">LGPL-2.1</license>
    <notes>
+* [mms] Fix handling servers that don&apos;t support ESMTP (Bug #12853).
 * [mms] Fix returning reply code in Exceptions.
    </notes>
   </release>