Milan Oberkirch added the comment:
Should this task get closed? (see comment above)
--
nosy: +jesstess
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20903
___
Roundup Robot added the comment:
New changeset 6cd64ef6fc95 by R David Murray in branch '2.7':
#20903: clarify what happens when an smtp connection timeout occurs.
http://hg.python.org/cpython/rev/6cd64ef6fc95
New changeset ca88bcfa5c15 by R David Murray in branch '3.4':
#20903: clarify what
R. David Murray added the comment:
Well, now that I applied your patch it can be :) Thanks.
--
resolution: - fixed
stage: needs patch - resolved
status: open - closed
versions: -Python 3.3
___
Python tracker rep...@bugs.python.org
Vyacheslav Rafalskiy added the comment:
Take look at this one too: http://bugs.python.org/issue2118
It is supposed to be fixed.
--
nosy: +Vyacheslav.Rafalskiy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20903
Milan Oberkirch added the comment:
The attached patch mentions the exception besides the description of the
timeout parameter. It doesn't hurt and makes programmers aware of this common
exception.
--
keywords: +patch
nosy: +zvyn
Added file:
New submission from Chris Clark:
The documentation for smtplib.SMTP says If the connect() call returns anything
other than a success code, an SMTPConnectError is raised. It doesn't
explicitly specify what happens when connect() raises instead of returns, but I
think either the documentation
R. David Murray added the comment:
In general we don't document what gets raised unless it is unique to a module
or has non-obvious implications. In this case, getting socket exceptions out
of a network library should be obvious in some sense :). In particular, both
SMTPConnectError and
Chris Clark added the comment:
I am concerned about the policy of not documenting all exceptions that are
raised. It sounds like there is no straightforward way to write a comprehensive
except statement without using a bare except or catching some base exception. I
consider it dangerous to
