Skip to content

Commit

Permalink
Cosmetic smtplib changes (pythonGH-8718)
Browse files Browse the repository at this point in the history
Some cosmetic smtplib changes here. Let me know if you'd like a bpo/news entry or splitting this PR into two.

Automerge-Triggered-By: @maxking
  • Loading branch information
scop committed May 23, 2020
1 parent 6fad3e6 commit da51ba4
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
7 changes: 4 additions & 3 deletions Doc/library/smtplib.rst
Expand Up @@ -279,9 +279,10 @@ An :class:`SMTP` instance has the following methods:
response for ESMTP option and store them for use by :meth:`has_extn`.
Also sets several informational attributes: the message returned by
the server is stored as the :attr:`ehlo_resp` attribute, :attr:`does_esmtp`
is set to true or false depending on whether the server supports ESMTP, and
:attr:`esmtp_features` will be a dictionary containing the names of the
SMTP service extensions this server supports, and their parameters (if any).
is set to ``True`` or ``False`` depending on whether the server supports
ESMTP, and :attr:`esmtp_features` will be a dictionary containing the names
of the SMTP service extensions this server supports, and their parameters
(if any).

Unless you wish to use :meth:`has_extn` before sending mail, it should not be
necessary to call this method explicitly. It will be implicitly called by
Expand Down
6 changes: 3 additions & 3 deletions Lib/smtplib.py
Expand Up @@ -222,7 +222,7 @@ class SMTP:
helo_resp = None
ehlo_msg = "ehlo"
ehlo_resp = None
does_esmtp = 0
does_esmtp = False
default_port = SMTP_PORT

def __init__(self, host='', port=0, local_hostname=None,
Expand Down Expand Up @@ -452,7 +452,7 @@ def ehlo(self, name=''):
self.ehlo_resp = msg
if code != 250:
return (code, msg)
self.does_esmtp = 1
self.does_esmtp = True
#parse the ehlo response -ddm
assert isinstance(self.ehlo_resp, bytes), repr(self.ehlo_resp)
resp = self.ehlo_resp.decode("latin-1").split('\n')
Expand Down Expand Up @@ -781,7 +781,7 @@ def starttls(self, keyfile=None, certfile=None, context=None):
self.helo_resp = None
self.ehlo_resp = None
self.esmtp_features = {}
self.does_esmtp = 0
self.does_esmtp = False
else:
# RFC 3207:
# 501 Syntax error (no parameters allowed)
Expand Down

0 comments on commit da51ba4

Please sign in to comment.