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

Pboucherat/001 add nxp and hsm support #23

Conversation

pboucherat-thegoodpenguin
Copy link
Contributor

Adds support for YubiHSM 2 hardware token for doing i.MX code signing

Save input file under its own name to match what is likely in any csf
file, can then use the same csf file on or offline.

[Original author vs v0.10.1]
Signed-off-by: Rob Ellis <rellis@thegoodpenguin.co.uk>
[Move to v0.12.0]
Signed-off-by: Phil Boucherat <pboucherat@thegoodpenguin.co.uk>
NXP CST tools added to docker image plus support for YubiHSM 2 hardware
token and pkcs11 so private keys for i.MX signing can be kept on the token.

[Added NXP CST tools to docker image]
Signed-off-by: Rob Ellis <rellis@thegoodpenguin.co.uk>
[Added YubiHSM and pkcs11 support to docker image]
Signed-off-by: Phil Boucherat <pboucherat@thegoodpenguin.co.uk>
Add a parameter to the /sign/imx API to select between the ssl and
pkcs11 backends for the NXP code signing tool.

Signed-off-by: Phil Boucherat <pboucherat@thegoodpenguin.co.uk>
Signed-off-by: Phil Boucherat <pboucherat@thegoodpenguin.co.uk>
@pboucherat-thegoodpenguin
Copy link
Contributor Author

Commit 915d9bd is not necessary, only a convenience thing while developing/testing I think.

@rellis-thegoodpenguin
Copy link

Commit 915d9bd stops things relying on a user knowing you have to set the filename in the csf to artifact and one can then use the same csf file on or offline without modification.

@pboucherat-thegoodpenguin
Copy link
Contributor Author

Ok, that makes sense.

@madisongh
Copy link
Owner

Thanks!

@madisongh madisongh merged commit cc60c89 into madisongh:master Dec 17, 2023
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

Successfully merging this pull request may close these issues.

None yet

3 participants