-
Notifications
You must be signed in to change notification settings - Fork 4.6k
CLI: Support dumping the TX message in sign-only mode #15834
Conversation
6136416
to
1e6994d
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
How do you want to handle updating spl-token-cli? |
1e6994d
to
0030b47
Compare
Pull request has been modified.
I'll do it once 1.5.15 crates are live. So much less plumbing there. The itch to refactor CLI similarly is getting strong now |
Ya, just don't think this will pass downstream-projects due to the |
Countdown to SPL in monorepo... |
0030b47
to
8f7a790
Compare
I'm adding a |
8f7a790
to
d82ac80
Compare
d82ac80
to
ed4f155
Compare
Codecov Report
@@ Coverage Diff @@
## master #15834 +/- ##
=========================================
- Coverage 80.0% 79.9% -0.1%
=========================================
Files 409 409
Lines 106389 106636 +247
=========================================
+ Hits 85163 85303 +140
- Misses 21226 21333 +107 |
Problem
No way to output the binary transaction message with CLI
Summary of Changes
Add a flag to output the base64-encoded transaction message in sign-only mode
Fixes #15786