[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2016-09-08 Thread Christian Heimes
Christian Heimes added the comment: starttls() does accept a ssl_context argument for a while. -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-09-01 Thread Michele Orrù
Michele Orrù added the comment: R. David Murray rep...@bugs.python.org writes: Since we want to encourage people to use the context, that sounds reasonable for 3.x at least. Concerning this specific proposition, I really don't see the point in having .starttls() not simply accepting a

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-08-29 Thread Michele Orrù
New submission from Michele Orrù: maker hello! In https://docs.python.org/2/library/smtplib.html#smtplib.SMTP.starttls I read:: maker If keyfile and certfile are provided, these are passed to the socket module’s ssl() function. maker socket.ssl() exists, though it is not documented (not even

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-08-29 Thread Alex Gaynor
Alex Gaynor added the comment: Attached patch fixes this up. -- keywords: +needs review, patch nosy: +alex Added file: http://bugs.python.org/file36505/t22301.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22301

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-08-29 Thread Terry J. Reedy
Changes by Terry J. Reedy tjre...@udel.edu: -- stage: - patch review versions: -Python 3.1, Python 3.2, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22301 ___

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-08-29 Thread Antoine Pitrou
Antoine Pitrou added the comment: Or perhaps we should remove the function to wrap_socket(), which is just an implementation detail? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22301

[issue22301] smtplib.SMTP.starttls' documentation is just confusing

2014-08-29 Thread R. David Murray
R. David Murray added the comment: Remove the reference, you mean? As in just delete the confusing line? Since we want to encourage people to use the context, that sounds reasonable for 3.x at least. -- ___ Python tracker rep...@bugs.python.org