Skip to content

Remove certificates, they are obsolete and are out of sync #131

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

Merged
merged 5 commits into from
Apr 28, 2022

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Apr 26, 2022

This Cert has been removed from the NINA firmware long time ago arduino/nina-fw@53580a0#diff-88545c89de17b94ff13f555cea95c4971daff07b1e887f1acb0811e7a674347cL545-L572. Furthermore, is about to expire.

The certificates under cert/ folder are there only there because if the user decides to upload a new self-signed cert, the partition inside the NINA module containing the certs gets wiped out, so it's required to upload the cert with a command line containing all the cert the user wants on the board. https://arduino.github.io/arduino-fwuploader/dev/usage/#certificates

This Cert has been removed from the NINA firmware long time ago arduino/nina-fw@53580a0#diff-88545c89de17b94ff13f555cea95c4971daff07b1e887f1acb0811e7a674347cL545-L572. Furthermore, is about to expire.
@umbynos umbynos added the topic: infrastructure Related to project infrastructure label Apr 26, 2022
@umbynos umbynos self-assigned this Apr 26, 2022
@umbynos
Copy link
Contributor Author

umbynos commented Apr 27, 2022

Actually, all the certs present in the certs folder are out of sync with the ones bundled in the nina fw. So I'm manually mirroring that ones to this repo too. Support for PEM encoded certs is in #132.

@umbynos
Copy link
Contributor Author

umbynos commented Apr 27, 2022

The chek-certificates workflow needs to be adjusted because it does not expect a single bundle of certificates.

@umbynos umbynos force-pushed the umbynos/remove_geotrust_cert branch from 0a74467 to b9acf06 Compare April 28, 2022 11:38
@umbynos umbynos force-pushed the umbynos/remove_geotrust_cert branch from b9acf06 to e629475 Compare April 28, 2022 11:42
@umbynos umbynos changed the title Delete GeoTrustGlobalCA_Root.cer Remove certificates, they are obsolete and are out of sync Apr 28, 2022
@umbynos
Copy link
Contributor Author

umbynos commented Apr 28, 2022

After a discussion with @per1234 we decided it was better to remove completely the certs and update the documentation instead. The check certificates workflow is not useful anymore now.

@umbynos umbynos merged commit ba458a2 into main Apr 28, 2022
@umbynos umbynos deleted the umbynos/remove_geotrust_cert branch April 28, 2022 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants