Message 250741 - Python tracker

➜

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.

In-reply-to
Author	vstinner
Recipients	christian.heimes, gbremer, vstinner
Date	2015年09月15日.08:43:17
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1442306597.29.0.036543212755.issue25115@psf.upfronthosting.co.za>

Content
+ if (depth < 0 \|\| depth > 100) { Why 100 and not 10 or 1000? SSL_CTX_set_verify_depth() is unable to check the depth? The patch lacks unit tests and documentation. The patch is for Python 2.7, it would be better to write a patch for the default branch (future Python 3.6).

Content

+ if (depth < 0 || depth > 100) {
Why 100 and not 10 or 1000?
SSL_CTX_set_verify_depth() is unable to check the depth?
The patch lacks unit tests and documentation.
The patch is for Python 2.7, it would be better to write a patch for the default branch (future Python 3.6).

History
Date	User	Action	Args
2015年09月15日 08:43:17	vstinner	set	recipients: + vstinner, christian.heimes, gbremer
2015年09月15日 08:43:17	vstinner	set	messageid: <1442306597.29.0.036543212755.issue25115@psf.upfronthosting.co.za>
2015年09月15日 08:43:17	vstinner	link	issue25115 messages
2015年09月15日 08:43:17	vstinner	create

homepage