-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add support for signing with an external program #312
Conversation
21800b5
to
62c33bf
Compare
Also one more suggestion: |
62c33bf
to
dadf72c
Compare
Thanks for catching the two issues. I've fixed them and also updated the e2e tests to test and validate this new functionality. |
dadf72c
to
4686fdd
Compare
Hmm, the padding I'm adding does not match what avbtool does. Looking into it now. EDIT: Found the issue, I'm an idiot. Fixing it now. |
Issue: #312 Signed-off-by: Andrew Gunnerson <[email protected]>
4686fdd
to
abdbd6f
Compare
Issue: #312 Signed-off-by: Andrew Gunnerson <[email protected]>
abdbd6f
to
672bccc
Compare
The padding has been fixed and I've added a new |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for now, tested on an actual device and seems to be working (at least for the AVB part)
By default, the helper program is invoked in a way that is compatible with avbtool's --signing_helper. However, the arguments have been extended slightly to allow passing in the passphrase file or environment variable for non-interactive use. Fixes: #310 Signed-off-by: Andrew Gunnerson <[email protected]>
Issue: #312 Signed-off-by: Andrew Gunnerson <[email protected]>
672bccc
to
031ac8a
Compare
Thanks for reviewing and testing! I made one final minor change to the README to improve the wording. I'll merge this once the CI builds complete. |
Signed-off-by: Andrew Gunnerson <[email protected]>
By default, the helper program is invoked in a way that is compatible with avbtool's
--signing_helper
. However, the arguments have been extended slightly to allow passing in the passphrase file or environment variable for non-interactive use.Fixes: #310