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

signing key changed #1

Open
obfusk opened this issue Sep 9, 2023 · 5 comments
Open

signing key changed #1

obfusk opened this issue Sep 9, 2023 · 5 comments

Comments

@obfusk
Copy link

obfusk commented Sep 9, 2023

The F-Droid build for v1.0.6 failed because the signing key changed.

It looks like v1.0.4 was signed with a debug key by accident (which unfortunately we seem to have overlooked on our end) and you switched to a proper release key after that?

org.sirekanyan.knigopis-1.0.4-36-release.apk
Signer #1 certificate DN: C=US, O=Android, CN=Android Debug
Signer #1 certificate SHA-256 digest: 9090b45fc82ff715c4f1b7162f79fb3a69f3b00c28a7a1b13be603dba3fa9ca0

org.sirekanyan.knigopis-1.0.5-37-release.apk
Signer #1 certificate DN: CN=Vadik Sirekanyan, OU=Software Development, O=sirekanyan.org, L=Yerevan, ST=Yerevan, C=AM
Signer #1 certificate SHA-256 digest: 613948a35cdbe1d151954bfccb66eb74c830ea69242b308a44ddb55aacae4f2c

org.sirekanyan.knigopis-1.0.6-38-release.apk
Signer #1 certificate DN: CN=Vadik Sirekanyan, OU=Software Development, O=sirekanyan.org, L=Yerevan, ST=Yerevan, C=AM
Signer #1 certificate SHA-256 digest: 613948a35cdbe1d151954bfccb66eb74c830ea69242b308a44ddb55aacae4f2c

org.sirekanyan.knigopis-1.0.7-39-fdroid-release.apk
Signer #1 certificate DN: CN=Vadik Sirekanyan, OU=Software Development, O=sirekanyan.org, L=Yerevan, ST=Yerevan, C=AM
Signer #1 certificate SHA-256 digest: 613948a35cdbe1d151954bfccb66eb74c830ea69242b308a44ddb55aacae4f2c

cc @licaon-kter @IzzySoft

@IzzySoft
Copy link

IzzySoft commented Sep 9, 2023

@sirekanian I don't see any release notes about a key change. It's of course the right move to use a release key (and not a debug one) – but as you see, a key change causes build failures, so nobody who installed a previous version receives any updates until the builds are fixed, and even not afterwards (as a change of the signing key means one has to uninstall and re-install in order to get the new version).

As for us: we need to look to get a check established warning us in such a case (as this one here shows, it's not always that obvious as a debug key does not automatically imply a debug build).

As for you, @sirekanian – please see How to keep your key safe and what measures to take for the event of loss? Though it might not be "loss" here, the measures taken in case of are comparable. E.g. to confirm it was really you it would help to have a copy of one of the newer releases (i.e. an APK built from the same commit as e.g. v1.0.7) signed with the "old key". And there should be a hint with the release notes that a re-install is needed (generally it would be a good idea to have release notes; you've stopped updating yours in Triple-T after v1.0.5, which was the first affected one, and even for that there's no hint on the key change.

PS: Apologies if this sounds like a rant or even "scolding", it's not meant that way. It's meant as hints for improvements – and request for confirmation. Thanks!

@licaon-kter
Copy link

licaon-kter commented Sep 12, 2023

@IzzySoft
Copy link

So have we proof fir the legitimacy of the new key meanwhile, @licaon-kter? I see no "reference APK" signed with the previous key at the release so we could check, as I suggested in my previous comment – nor any reference to a verification having taken place.

@licaon-kter
Copy link

@IzzySoft upstream did not interact here yet so 🤷

@IzzySoft
Copy link

upstream did not interact here yet so 🤷

So disable updates for now. If upstream does not even respond to a security issue after almost a full week, we must consider the repo compromised. @sirekanian (or their account) was active every day since (with the exception of September 11th) with multiple contributions each day, so "AFK" cannot be the reason. I see in your comment above you already did that, thanks! I've just added a comment to both commits for reference.

@sirekanian until this issue here is solved and we were able to verify the new signing key is legit, your app will not be available at F-Droid: all its versions have been disabled, and auto-update as well. Your turn now.

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

No branches or pull requests

3 participants