Skip to content

Include hash of pip wheel in ensurepip #112721

Closed

Description

@sethmlarson

Feature or enhancement

Proposal:

Include the SHA 256 hash of the pip wheel artifact used in ensurepip module and fail execution if the checksum doesn't match. This will serve as a protection mechanism allowing non-trusted contributors to contribute updates to artifacts and allow reviewers to quickly check in the PyPI UI whether the artifact has the correct checksum.

This feature request was spawned from this PR: #112517 which required a bit more reviewing work to be confident in the contributed artifact.

Has this already been discussed elsewhere?

I have already discussed this feature proposal on Discourse

Links to previous discussion of this feature:

Discussed in the release Discord channel.

Activity

  1. pfmoore commented on Dec 4, 2023

    @pfmoore
    Member

    Note that this would not imply that we should accept ensurepip upgrades from anyone other than the pip maintainers, so that we (pip) can delay pushing a pip release to ensurepip if we ever need to.

  2. pradyunsg commented on Dec 4, 2023

    @pradyunsg
    Member

    This will serve as a protection mechanism allowing non-trusted contributors to contribute updates to artifacts and allow reviewers to quickly check in the PyPI UI whether the artifact has the correct checksum.

    We already have a protection mechanism in place in the form of a CI check.

    https://github.com/python/cpython/blob/main/.github/workflows/verify-ensurepip-wheels.yml

  3. sethmlarson commented on Dec 4, 2023

    @sethmlarson
    ContributorAuthor

    Oh cool, this issue is probably not needed then. Thanks @pradyunsg!

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

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions