[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-28 Thread Giovanni Cappellotto
Change by Giovanni Cappellotto : -- nosy: +potomak ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-24 Thread Senthil Kumaran
Senthil Kumaran added the comment: Suggesting the use of context and deprecating `cafile=None, capath=None, cadefault=False` sounds like a good idea. -- ___ Python tracker

[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-24 Thread Karthikeyan Singaravelan
Change by Karthikeyan Singaravelan : -- nosy: +orsenthil ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-24 Thread Christian Heimes
Christian Heimes added the comment: Yes, that's my favorite solution. Actually I would like to deprecate and remove all TLS/SSL related arguments in favor of a single context/ssl_context argument. -- ___ Python tracker

[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-23 Thread Gregory P. Smith
Gregory P. Smith added the comment: Probably nevermind on implementing this: ssl_context = ssl.create_default_context(cadata=...) urllib.request.urlopen(..., context=ssl_context) Works fine. It might be worth a note in the urlopen() docs if anything. --

[issue37666] urllib.requests.urlopen doesn't support cadata=

2019-07-23 Thread Gregory P. Smith
New submission from Gregory P. Smith : The underlying https://docs.python.org/3/library/ssl.html#ssl.SSLContext.load_verify_locations API supports cafile, capath and cadata. urlopen() only offers up cafile and capath. It should also support cadata for completeness. This matters for