From: Barry Warsaw <barry@python.org>
Date: Mon, 15 Apr 2002 20:03:30 +0000 (+0000)
Subject: ehlo(): A proper fix for SF bug #498572.  RFC 1869 describes ESMTP
X-Git-Tag: v2.3c1~5943
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=be22ae6def4ff3bfb826a96c1e5b1360b5a7c1e0;p=thirdparty%2FPython%2Fcpython.git

ehlo(): A proper fix for SF bug #498572.  RFC 1869 describes ESMTP
which requires that if there are ehlo parameters returned with an ehlo
keyword (in the response to EHLO), the keyword and parameters must be
delimited by an ASCII space.  Thus responses like

    250-AUTH=LOGIN

should be ignored as non-conformant to the RFC (the `=' isn't allowed
in the ehlo keyword).

This is a bug fix candidate.
---

diff --git a/Lib/smtplib.py b/Lib/smtplib.py
index 6f37ceea1307..283616581fda 100755
--- a/Lib/smtplib.py
+++ b/Lib/smtplib.py
@@ -398,7 +398,11 @@ class SMTP:
         resp=self.ehlo_resp.split('\n')
         del resp[0]
         for each in resp:
-            m=re.match(r'(?P<feature>[A-Za-z0-9][A-Za-z0-9\-]*)',each)
+            # RFC 1869 requires a space between ehlo keyword and parameters.
+            # It's actually stricter, in that only spaces are allowed between
+            # parameters, but were not going to check for that here.  Note
+            # that the space isn't present if there are no parameters.
+            m=re.match(r'(?P<feature>[A-Za-z0-9][A-Za-z0-9\-]*) ?',each)
             if m:
                 feature=m.group("feature").lower()
                 params=m.string[m.end("feature"):].strip()