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

Build: Verify the Python checksums #2239

Merged
merged 1 commit into from Apr 9, 2021

Conversation

EchterAgo
Copy link

Before this change, whoever has a copy of the Python release signing key is a trusted third party. This commit changes this so that in addition to checking the signature of the installers, the checksums are also verified.

This is motivated by the Python 3.8.9 release (2021-04-02) being replaced with a different version (2021-04-06) that didn't match the PGP signature.

Before this change, whoever has a copy of the Python release signing key
is a trusted third party. This commit changes this so that in addition
to checking the signature of the installers, the checksums are also
verified.

This is motivated by the Python 3.8.9 release (2021-04-02) being
replaced with a different version (2021-04-06) that didn't match the
PGP signature.
@EchterAgo
Copy link
Author

The full build is still running, but I tested the checksum part, good checksums pass, bad checksums fail the build.

@EchterAgo
Copy link
Author

@cculianu
Copy link
Collaborator

cculianu commented Apr 9, 2021

Yes, this is a good idea.

@cculianu cculianu merged commit af09280 into Electron-Cash:master Apr 9, 2021
@EchterAgo EchterAgo deleted the verify_python_checksum branch April 9, 2021 12:36
@SomberNight
Copy link

Decent change but the true fix to remove trust would be to build python from source in the script :/

@EchterAgo
Copy link
Author

Decent change but the true fix to remove trust would be to build python from source in the script :/

Yea, right. I'm not sure if the Windows Python Installer is built reproducible. Now that https://bugs.winehq.org/show_bug.cgi?id=49897 is fixed in Wine 6.6, it should be possible to get the Visual C++ Build Tools working more easily in Wine, so maybe we can build Python and extensions soon.

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.

None yet

3 participants