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

Feature/New tool: Arifact signing tool #147

Closed

Conversation

minosgalanakis
Copy link

@minosgalanakis minosgalanakis commented Jul 9, 2024

This pr is introducing the artifact sign tool, as discussed in the design review meeting.

It is a standalone tool serving a web-app and using gpg tool to sign and validate release tarballs.

How to test locally:

cd tools/artifact_sign
python -m vevn artf_env && artf_env/bin/activate && pip install artf_sign_requirements.txt
MBEDTLS_FW_SIGN_SERVER_IP=X.X.X.X && python artifact_sign.py

@minosgalanakis minosgalanakis added enhancement New feature or request size-s Estimated task size: small (~2d) priority-medium labels Jul 9, 2024
@minosgalanakis minosgalanakis force-pushed the feature/fw_signer branch 2 times, most recently from 5b9fa5d to 184858c Compare July 9, 2024 14:24
This pr introduces a simple web-app to sign
firmware-files using gpg.

Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
@minosgalanakis minosgalanakis changed the title Feature/New tool: Firmware signing tool Feature/New tool: Arifact signing tool Jul 9, 2024
Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Since this is supposed to be on the critical path for releases, it needs to be held to a high standard of review. So it needs to be either in https://github.com/Mbed-TLS/mbedtls-framework or https://github.com/Mbed-TLS/mbedtls-test.
  2. Please rename it to “release signing tool”. “Artifact” is not a word we use elsewhere and you didn't even spell it consistently.

@minosgalanakis
Copy link
Author

Moved to Mbed-TLS/mbedtls-test#173

@minosgalanakis minosgalanakis deleted the feature/fw_signer branch August 7, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-medium size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants