-
Notifications
You must be signed in to change notification settings - Fork 11
Async and support TAPSIGNER backup and change commands, sign PSBT
#29
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
Merged
Merged
Changes from 30 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
0450bc6
Update deps and fix breaking changes
praveenperera 0ef500f
Make the `CkTransport` trait `async`
praveenperera e6fa7ab
Fix CLI
praveenperera 42a04fa
Allow transmit to be used in async contexts
praveenperera 5dc76a5
Separate out functions for `TAPSIGNER` & `SATSCARD`
praveenperera b51dd5b
Separate apdu commands to modules
praveenperera 0440250
Change all `cmd` from `String` to `&'static str`
praveenperera 591f92a
Change all `epubkey` to be 33 bytes
praveenperera 2756692
Create new `TapSignerError`
praveenperera 115374d
Set card nonce as 16 byte array
praveenperera d1a7bc6
Implement `change` and `backup` commands and validation logic
praveenperera a81474f
Add `backup` and `change` commands to CLI
praveenperera d36a8f0
Complete `change` command
praveenperera bf32158
Improve error handling
praveenperera fe3033e
Fix imports
praveenperera a74be41
Rework CvcChangeError
praveenperera 2f8d83e
Make `chain_code` a byte array instead of a Vec
praveenperera c754df5
Make `derive` signature verification work with a derivation path
praveenperera 22515cc
Use slice instead of Vec for master pubkey
praveenperera dd01257
Create `CkTapError` enum to represent errors returned by the card
praveenperera a863a0a
Merge pull request #1 from bitcoinppl/tapsigner-change
praveenperera cf7f6e2
Adjust wording on `BackupFirst` error
praveenperera 37721f4
Fix signature verification for `derive` command
praveenperera 3dfec7e
Added note about signature verification for derive command
praveenperera f6841a3
Revert changes, SATSCARD won't have pubkey set
praveenperera b020c4d
Update tap_signer.rs
praveenperera 98c21e0
Add `StatusCommand` to TapSigner
praveenperera b787752
Update tap_signer.rs
praveenperera f93d247
Fix errors in examples
praveenperera 54bd325
Make all commands take a &str instead of a String for cvc and fix tests
praveenperera c907f6f
Add `psbt` signing to lib
praveenperera ae2fb07
Use byte arrays whenever possible
praveenperera 898f7be
Add `sign` command to cli
praveenperera 2097a84
Use a `vec` for subpath because it can be 0,1,2 in length
praveenperera 4856cc0
Fix typo in comments
praveenperera ab0ae62
Make `bitcoin` a regular dependency
praveenperera 0b98baa
Add `reason` to `InvalidScript` error
praveenperera ad81034
Retry `UnluckyNumber` errors during `sign` command
praveenperera bbcb1d2
Use secp from `bitcoin`, verify pubkey matches
praveenperera c71ba3f
Add function to finalize PSBT
praveenperera 37f7fc6
Use byte arrays for `DeriveResponse`
praveenperera 2871698
Get the subpath correctly from the PSBT
praveenperera fe190c4
Add docs and remove finalizing PSBT, leave it up to the user
praveenperera 7db62ee
Fix clippy warnings
praveenperera 30bfddb
Improve comments on the PSBT signing code
praveenperera 9dfa48d
Merge pull request #2 from bitcoinppl/sign-bitcoin-txn
praveenperera 09c2f57
Derive pubkey from TAPSIGNER if we get a pubkey mismatch
praveenperera 9cf9fbd
Get the correct path for deriving
praveenperera 3ac5f95
Add note about toggling the hardened bit
praveenperera 1f95e27
Update tap_signer.rs
praveenperera 6609c34
Update README.md to show that we've added support for `change` and `b…
praveenperera 3d7c1ec
Make uniffi dep version less restrictive
praveenperera File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.