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

Adding Hash of Signing Key to README #11140

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

shrimprugbysnowowl
Copy link

What is it?

  • Simple Security Addition

Description of the changes in your PR

For your consideration, I'm submitting a small edit to the README to add the hash of the signing key currently posted on the NewPipe website and bring it to the user's attention in installation method #2.

For android users looking to download the APK and install directly from Github, or using a different method like Obtainium, the responsibility of verifying the signing key of the downloaded APK is placed on the user. This is easily done with AppVerifier. A signing certificate posted on both the website and in the README would improve user confidence in the download.

Thank you for your consideration.

@github-actions github-actions bot added the size/small PRs with less than 50 changed lines label Jun 7, 2024
@opusforlife2
Copy link
Collaborator

This will have to be kept track of and updated on every new release. Is there no way to automate this?

@shrimprugbysnowowl
Copy link
Author

shrimprugbysnowowl commented Jun 7, 2024 via email

@opusforlife2
Copy link
Collaborator

Oh, my bad, I was thinking of the hash of the APK.

@AudricV AudricV added the meta Related to the project but not strictly to code label Jun 7, 2024
@shrimprugbysnowowl
Copy link
Author

shrimprugbysnowowl commented Jun 7, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Related to the project but not strictly to code size/small PRs with less than 50 changed lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants