Skip to content

[3.5] bpo-36816: Update the self-signed.pythontest.net cert (GH-13192)#13200

Merged
larryhastings merged 3 commits into
python:3.5from
gpshead:backport-6bd8173-3.5
Jul 13, 2019
Merged

[3.5] bpo-36816: Update the self-signed.pythontest.net cert (GH-13192)#13200
larryhastings merged 3 commits into
python:3.5from
gpshead:backport-6bd8173-3.5

Conversation

@gpshead

@gpshead gpshead commented May 8, 2019

Copy link
Copy Markdown
Member

We updated the server, our testsuite must match.

https://bugs.python.org/issue36816

✈️ CLE -> DEN -> SFO ✈️ #pycon2019 #smoothjazz
(cherry picked from commit 6bd8173)

Authored-by: Gregory P. Smith greg@krypto.org

https://bugs.python.org/issue36816

…-13192)

We updated the server, our testsuite must match.

https://bugs.python.org/issue36816

✈️ CLE -> DEN ✈️ GH-pycon2019
(cherry picked from commit 6bd8173)

Co-authored-by: Gregory P. Smith <greg@krypto.org>
The 3.5 tree also needed a certificate in the capath directory updated.
The filename for that was determined by `openssl x509 -in $cert.pem -subject_hash`.
@gpshead

gpshead commented May 9, 2019

Copy link
Copy Markdown
Member Author

without this PR, python3.5 -m test.regrtest -u all test_httplib test_ssl will fail due to a certificate mismatch vs the updated cert on our server.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

This PR is needed for PR #13207, an important security fix: prevent HTTP Header Injection.

@larryhastings larryhastings merged commit 221178a into python:3.5 Jul 13, 2019
@bedevere-bot

Copy link
Copy Markdown

@larryhastings: Please replace # with GH- in the commit message next time. Thanks!

@larryhastings

Copy link
Copy Markdown
Contributor

Thanks for the 3.5 love!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sprint tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants