[issue10852] SSL/TLS sni use in smtp, pop, imap, nntp, ftp client libs by default

2012-08-20 Thread danblack
danblack added the comment: Antoine, I copied off your http example for all the other protocols. tested with: import smtplib a = smtplib.SMTP_SSL('gmail-smtp-in.l.google.com.') a.starttls() a = smtplib.SMTP_SSL('mail.internode.on.net') a = smtplib.SMTP_SSL('smtp.g

[issue10852] SSL/TLS sni use in smtp, pop, imap, nntp, ftp client libs by default

2012-08-21 Thread danblack
Changes by danblack : Removed file: http://bugs.python.org/file26918/issue_10852_pop-smtp-imap-nntp.patch ___ Python tracker <http://bugs.python.org/issue10852> ___ ___

[issue10852] SSL/TLS sni use in smtp, pop, imap, nntp, ftp client libs by default

2012-08-21 Thread danblack
danblack added the comment: previous patch had dumb error and even failed test suit. Now fixed. -- Added file: http://bugs.python.org/file26949/issue10852-sni.patch ___ Python tracker <http://bugs.python.org/issue10

[issue8109] Server-side support for TLS Server Name Indication extension

2012-08-21 Thread danblack
danblack added the comment: test_sni not working. getpeercert() not returning a certificate. -- keywords: +patch nosy: +daniel-black Added file: http://bugs.python.org/file26950/issue8109_server_side_sni.patch ___ Python tracker <h

[issue10852] SSL/TLS sni use in smtp, pop, imap, nntp, ftp client libs by default

2012-08-22 Thread danblack
danblack added the comment: > Thanks for the patch > Daniel. 3.3 is nearing the release candidate phase, so I'm re-targetting to > 3.4. I'll take a detailed look soon. Welcome. Just noticed conflicts with #4473 in the client POP implementation. Hopefully they are close an

[issue8109] Server-side support for TLS Server Name Indication extension

2012-09-14 Thread danblack
Changes by danblack : Removed file: http://bugs.python.org/file26950/issue8109_server_side_sni.patch ___ Python tracker <http://bugs.python.org/issue8109> ___ ___ Pytho

[issue8109] Server-side support for TLS Server Name Indication extension

2012-09-14 Thread danblack
danblack added the comment: > Daniel, your patch looks quite interesting. Please, send a contributor > agreement to the PSF: http://www.python.org/psf/contrib/contrib-form-python/ > . Let me know when you status have changed. Already done. Has been accepted and I've got an a

[issue8109] Server-side support for TLS Server Name Indication extension

2012-10-05 Thread danblack
danblack added the comment: happy with this? I'm not sure what i've done to make s._set_context(newctx) work but s.context = newctx fail. I though the code here http://bugs.python.org/review/8109/diff2/5815:5989/Lib/ssl.py effectively

[issue8109] Server-side support for TLS Server Name Indication extension

2012-12-01 Thread danblack
danblack added the comment: > Antoine Pitrou (pitrou) * Date: 2012-10-06 13:10 > Daniel, I'll take a look. minor nag :-) -- ___ Python tracker <http://bugs.python

[issue8109] Server-side support for TLS Server Name Indication extension

2012-12-09 Thread danblack
danblack added the comment: I've added a full set of alert descriptions and cleaned up the doco some more. The reference counting when the SNI callback comes in is my greatest worry. -- Added file: http://bugs.python.org/file28271/issue-8109-sni-serverside.

[issue10852] SSL/TLS sni use in smtp, pop, imap, nntp, ftp client libs by default

2012-12-09 Thread danblack
danblack added the comment: the one error in the previous review corrected. -- ___ Python tracker <http://bugs.python.org/issue10852> ___ ___ Python-bugs-list m

[issue8109] Server-side support for TLS Server Name Indication extension

2012-12-15 Thread danblack
Changes by danblack : Added file: http://bugs.python.org/file28326/issue-8109-sni-serverside.patch ___ Python tracker <http://bugs.python.org/issue8109> ___ ___ Python-bug

[issue8109] Server-side support for TLS Server Name Indication extension

2012-12-16 Thread danblack
danblack added the comment: > If it isn't very clear to you, I can still handle it myself, though. Those > docs aren't the best. Not clear enough. Yes I'd appreciate you handling it. Thanks. -- ___ Python tracker <http://b

[issue8109] Server-side support for TLS Server Name Indication extension

2013-01-05 Thread danblack
danblack added the comment: > I've committed the latest patch. Thank you very much! much appreciate your help. -- ___ Python tracker <http://bugs.python.or

[issue8109] Server-side support for TLS Server Name Indication extension

2013-01-05 Thread danblack
danblack added the comment: > I don't know which error code should be returned in this case. Thanks Christian. My fault - asked Antoine to remove the default value for it and didn't see this like. make line 2403: return SSL_