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

Use cacert.pem from certifi 2024.7.4. #14

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

jsirois
Copy link
Member

@jsirois jsirois commented Jul 25, 2024

Add nox -e update-certifi-cacert-pem and run it to refresh Pip's
vendored certifi with the latest CA certs that should be trusted.

CF #12 by @gs-kamnas that prompted this.

This allows updating Pex's vendored Pip with the latest CA certs from
certifi without updating Pip's vendored certifi code otherwise. This is
useful since certifi has dropped support for Pythons Pex supports, but
the `cacert.pem` file it uses has a stable format across all certifi
versions.
@gs-kamnas
Copy link

This looks reasonable from an implementation and security standpoint as well. Valid point with regards to compatibility limitations with older python interpreter versions in my implementation in #12.

@jsirois
Copy link
Member Author

jsirois commented Jul 25, 2024

@gs-kamnas thanks for prompting this long overdue refresh. I'm sorry I didn't notice your PR until just a few days ago! Pex vendored this old Pip a good while ago and so I ~never look back here, but was prompted by the fix for statically compiled CPython support.

@benjyw I've tested this out over in a Pex branch and the comment I add at the top of the cacert.pem causes no issues; so I'll submit and fire up a Pex change to get out this security fix.

@gs-kamnas
Copy link

Appears that the testsuite also requires making calls to Git as part of execution.

Therefore certifi/refresh-cacert.pem...gs-kamnas:pex-pip:certifi/refresh-cacert.pem may help with a few of the testcases that are presently failing.

@jsirois
Copy link
Member Author

jsirois commented Jul 25, 2024

@gs-kamnas - yeah - these are the Pip projects CI setups which I haven't tried to get / keep working in the fork. My ground truth CI for any edits I make in this branch are back in pex-tool/pex where it heavily exercises Pip 20.3.4+patched (its vendored Pip that comes from this branch).

@jsirois jsirois merged commit 4850833 into pex/patches/generation-2 Jul 25, 2024
1 of 7 checks passed
@jsirois jsirois deleted the certifi/refresh-cacert.pem branch July 25, 2024 19:18
@jsirois
Copy link
Member Author

jsirois commented Jul 25, 2024

Ok, this is being used in pex-tool/pex#2476

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants