Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.12] gh-108303: Move all certificates to Lib/test/certdata/ (GH-109489) #109682

Merged
merged 6 commits into from
Oct 10, 2023

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 21, 2023

(cherry picked from commit e57ecf6)

Co-authored-by: Nikita Sobolev mail@sobolevn.me

…nGH-109489)

(cherry picked from commit e57ecf6)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM, good bot 🤖.

@gvanrossum
Copy link
Member

Is it really a good idea to backport moving files around?

@vstinner
Copy link
Member

@gvanrossum:

Is it really a good idea to backport moving files around?

I didn't want to backport these changes, but I was asked to backport them. See the discussion: #108303 (comment)

@vstinner
Copy link
Member

vstinner commented Oct 4, 2023

ERROR: test_starttls (test.test_nntplib.LocalServerTests.test_starttls)

Oh, all CIs failed on test_nntplib. I clicked on re-run failed jobs.

cc @sobolevn

@sobolevn
Copy link
Member

sobolevn commented Oct 9, 2023

@vstinner I fixed test_nntplib, it was removed in 3.13, so there was a difference.

@sobolevn
Copy link
Member

sobolevn commented Oct 9, 2023

Done! It works.

@vstinner vstinner merged commit c2d542b into python:3.12 Oct 10, 2023
@miss-islington miss-islington deleted the backport-e57ecf6-3.12 branch October 10, 2023 20:38
@vstinner vstinner added the needs backport to 3.11 only security fixes label Oct 10, 2023
@miss-islington
Copy link
Contributor Author

Thanks @miss-islington for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-app
Copy link

bedevere-app bot commented Oct 10, 2023

GH-110646 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Oct 10, 2023
miss-islington added a commit to miss-islington/cpython that referenced this pull request Oct 10, 2023
…pythonGH-109489) (pythonGH-109682)

* pythongh-108303: Move all certificates to `Lib/test/certdata/` (pythonGH-109489)
(cherry picked from commit e57ecf6)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>

Python 3.12 backport: update also `test_nntplib`.

(cherry picked from commit c2d542b)

Co-authored-by: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Co-authored-by: T. Wouters <thomas@python.org>
vstinner pushed a commit that referenced this pull request Oct 10, 2023
GH-109489) (GH-109682) (#110646)

[3.12] gh-108303: Move all certificates to `Lib/test/certdata/` (GH-109489) (GH-109682)

* gh-108303: Move all certificates to `Lib/test/certdata/` (GH-109489)
(cherry picked from commit e57ecf6)

Python 3.12 backport: update also `test_nntplib`.

(cherry picked from commit c2d542b)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Co-authored-by: T. Wouters <thomas@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants