This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author pitrou
Recipients barry, giampaolo.rodola, janssen, jcea, lcatucci, pitrou, r.david.murray
Date 2012-11-17.18:39:44
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1353177585.19.0.673877491498.issue4473@psf.upfronthosting.co.za>
In-reply-to
Content
Hello,

Here are some comments about the patches:

1) poplib_01_socket_shutdown_v3.diff: looks fine to me

2) poplib_02_server_capabilities_v3.diff:

- please try to follow PEP 8 (i.e. `capa = {}` not `capa={}`)
- I think the capa() result should be a dict mapping str keys to str values (not bytes), since that part of the POP3 protocol seems to have a well-defined character set (ASCII)

3) poplib_03_starttls_v3.diff:

- same comment about PEP 8
- why did you change the signature of the _create_socket() method? it looks gratuitous
- the new method should be called starttls() as in other modules, not stls()
- starttls() should only take a context argument; no need to support separate keyfile and certfile arguments
- what is the point of catching errors like this:

+        except error_proto as _err:
+            resp = _err
History
Date User Action Args
2012-11-17 18:39:45pitrousetrecipients: + pitrou, barry, jcea, janssen, giampaolo.rodola, lcatucci, r.david.murray
2012-11-17 18:39:45pitrousetmessageid: <1353177585.19.0.673877491498.issue4473@psf.upfronthosting.co.za>
2012-11-17 18:39:45pitroulinkissue4473 messages
2012-11-17 18:39:44pitroucreate