-
Notifications
You must be signed in to change notification settings - Fork 671
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
feat: add signer CLI function to generate signature #4319
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## next #4319 +/- ##
==========================================
- Coverage 83.35% 83.35% -0.01%
==========================================
Files 446 446
Lines 318545 318774 +229
==========================================
+ Hits 265539 265721 +182
- Misses 53006 53053 +47 ☔ View full report in Codecov by Sentry. |
ee8eabc
to
37c7c52
Compare
00d67fd
to
56c1dd6
Compare
381525b
to
53427cf
Compare
2b2b71a
to
aadf908
Compare
aadf908
to
7433864
Compare
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 :D Thankss for doing this!
|
||
#[derive(Parser, Debug, Clone)] | ||
/// Arguments for the generate-stacking-signature command | ||
pub struct GenerateStackingSignatureArgs { |
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.
It would be useful to have test vectors with hex strings, e.g. containing the serialized payload for this struct. That way other folks could leverage the same vectors to be more confident in their implementation (like in a js one).
Not super important though unless we actually envision needing to implement similar code in js any time soon.
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, thanks @hstove!
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.
Thanks for this @hstove! Is there a way to test this in an integration test, so as to verify that the output of this command can be used in pox-4
for a stacking signature?
Good call, added! |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This PR adds a new command to the
stacks-signer
CLI:This is built on top of #4277, which isn't merged yet, so I'm opening this as a draft until that PR is reviewed and merged.