urllib.request.urlopen should take a "context" (SSLContext) argument

[alex]

BPO	22366
Nosy	@ncoghlan, @orsenthil, @pitrou, @larryhastings, @giampaolo, @tiran, @alex, @dstufft
Files	urlopen-context.diff urlopen-context.diff urlopen-context.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/orsenthil'
closed_at = <Date 2014-09-19.07:26:02.958>
created_at = <Date 2014-09-08.17:45:52.591>
labels = ['library']
title = 'urllib.request.urlopen should take a "context" (SSLContext) argument'
updated_at = <Date 2014-11-24.04:58:39.680>
user = 'https://github.com/alex'

bugs.python.org fields:

activity = <Date 2014-11-24.04:58:39.680>
actor = 'benjamin.peterson'
assignee = 'orsenthil'
closed = True
closed_date = <Date 2014-09-19.07:26:02.958>
closer = 'orsenthil'
components = ['Library (Lib)']
creation = <Date 2014-09-08.17:45:52.591>
creator = 'alex'
dependencies = []
files = ['36573', '36576', '36580']
hgrepos = []
issue_num = 22366
keywords = ['patch', 'needs review']
message_count = 16.0
messages = ['226594', '226595', '226609', '226619', '226672', '226676', '227077', '227080', '227081', '227091', '227092', '227101', '227106', '230514', '230562', '231574']
nosy_count = 11.0
nosy_names = ['ncoghlan', 'janssen', 'orsenthil', 'pitrou', 'larry', 'giampaolo.rodola', 'christian.heimes', 'Arfrever', 'alex', 'python-dev', 'dstufft']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue22366'
versions = ['Python 2.7', 'Python 3.4', 'Python 3.5']

[alex]

Instead of the ca* arguments it currently takes, these can all be encapsulated into an SSLContext argument, which the underlying http.client already supports.

[alex]

Attached patch against the default branch adds support for this.

[alex]

Added a versionchanged directive to the docs.

[alex]

New patch switches to a ValueError and includes a test case that it's raised.

[orsenthil]

With the final review comment addressed, this could go in. Since you have commit rights, please feel free to commit it. (else, please assign this to me and I will commit and follow up with the buildbots)

[alex]

Replied to the review; let me know if you agree with my comment.

[alex]

Just wanted to ping you on this Senthil about committing this.

[python-dev]

New changeset c0b0dda16009 by Senthil Kumaran in branch 'default':
Issue bpo-22366: urllib.request.urlopen will accept a context object (SSLContext)
https://hg.python.org/cpython/rev/c0b0dda16009

[orsenthil]

Committed this in 3.5. (Since this is adding an new argument, it should not be backported to 3.4).

Thank you!

[larryhastings]

Alex said he needed this in order to implement PEP-476, and I thought PEP-476 was going in to 3.4 (hopefully for 3.4.2, which I'm tagging in like 24 hours).