frost/bdk
3
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
826 Commits 44 Branches 59 Tags
f3e479fa7f7240c888727ce2cf29f7f79fc4b2b3
Commit Graph
248 Commits
Author SHA1 Message Date
Steve Myers 1c6864aee8 Rename ToDescriptorKey to IntoDescriptorKey 2021-02-12 23:23:20 -08:00
Steve Myers d638da2f10 Rename ToWalletDescriptor to IntoWalletDescriptor 2021-02-12 23:23:20 -08:00
Steve Myers 04348d0090 Fix clippy warning 'wrong_self_convention' 2021-02-12 22:23:48 -08:00
Alekos Filini 3d9d6fee07 Update bitcoin, miniscript, electrum-client 2021-02-05 09:11:27 -05:00
LLFourn 6689384c8a Merge branch 'master' into make_txbuilder_take_ref_to_wallet 2021-01-30 13:12:13 +11:00
LLFourn dbf8cf7674 Make maintain_single_recipient return a Result 
preferable to panicking.
 2021-01-29 12:33:07 +11:00
Alekos Filini c431a60171 [signer] Add Signer::id() 
Closes #261
 2021-01-27 11:43:28 -05:00
LLFourn ff10aa5ceb Add "add_utxos" method on TxBuilder 
To replace the previously existing ".utxos"
 2021-01-22 15:08:31 +11:00
LLFourn 21d382315a Remove not very useful comment 
Thanks @tcharding.
 2021-01-22 15:08:30 +11:00
LLFourn 6fe3be0243 Derive Clone + Debug for TxBuilder 
And make Wallet Debug while I'm at it.
 2021-01-22 15:08:30 +11:00
LLFourn 10fcba9439 Revert back to Vec to hold utxos in builder 
Due to brain malfunction I made utxos into a BTree. This made a test
pass but is incorrect. The test itself was incorrect as per comment in

https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-758370380

So I (1) reverted utxos back to a Vec, (2) fixed the test and expanded
the comment in the test.
 2021-01-22 15:08:30 +11:00
LLFourn 890d6191a1 Remove Option trickery from TxBuilder API 
see: https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-754685962
 2021-01-22 15:08:30 +11:00
LLFourn 735db02850 Assert that .finish() hasn't been called already in coin_selection 2021-01-22 14:33:37 +11:00
LLFourn 7bf46c7d71 Add comment explaining why params and coin_selection are Options 2021-01-22 14:33:37 +11:00
LLFourn 8319b32466 Fix wrong doc links 2021-01-22 14:33:37 +11:00
LLFourn 5fb7fdffe1 [wallet] Use doctest_wallet!() to remove some no_runs from doctests 
...and improve the fee bumping example while trying to make it
no_run (but failed).
 2021-01-22 14:33:37 +11:00
LLFourn 7553b905c4 [wallet] Overhaul TxBuilder internals and externals 
Fixes #251

TxBuilders are now not created directly but are created through the
wallet with `build_tx` and `build_fee_bump`.
The advantages of this realised in this commit are:

1. Normal tx creation and fee bumping use the code internally. The only
difference between normal tx and fee bump is how the builder is created.
2. The TxBuilder now has a refernce to the wallet and can therefore
lookup things as methods are called on it. `add_utxo` now uses this to
look up UTXO deta when it is called (rather than having to do it and
possibly error later on).

To support these changes `get_utxo` and `get_descriptor_for_keychain`
public methods have been added to Wallet. I could have kept them
pub(crate) but they seem like fine APIs to have publicly.
 2021-01-22 14:33:37 +11:00
LLFourn f74f17e227 Change "received_tx" into "populate_test_db" macro 
A `[cfg(test)]` function is not as helpful as a macro since it can't be
called in the context of a doctest.

Also adds doctest_wallet macro which can be used to create a wallet in a
doctest.
 2021-01-22 14:23:36 +11:00
Justin Moon 7bc8c3c380 [wallet] Add "needed" and "available" metadata to Error::InsufficientFunds 2021-01-13 23:00:37 -06:00
LLFourn 0c9c0716a4 [wallet] Fix details.fees being wrong when change is dust 2020-12-29 16:36:35 +11:00
LLFourn 656c9c9da8 Use () to indicate a missing blockchain 
So that:
1. There are no runtime errors
2. There less type annotations needed
3. Less traits and stuff to document
 2020-12-23 14:52:29 +11:00
Alekos Filini 63aabe203f Merge commit 'refs/pull/235/head' of github.com:bitcoindevkit/bdk 2020-12-18 10:41:37 +01:00
Steve Myers 2325a1fcc2 [docs] Format code in docs with '--config format_code_in_doc_comments=true' 2020-12-16 15:12:51 -08:00
Alekos Filini 7e90657ee1 [descriptor] Make the syntax of descriptor!() more consistent 
The syntax now is pretty much the same as the normal descriptor syntax,
with the only difference that modifiers cannot be grouped together (i.e.
`sdv:older(144)` must be turned into `s:d:v:older(144)`.
 2020-12-16 19:00:55 +01:00
Alekos Filini 5f373180ff Merge commit 'refs/pull/223/head' of github.com:bitcoindevkit/bdk 2020-12-16 11:11:38 +01:00
Alekos Filini 931a110e4e Merge commit 'refs/pull/229/head' of github.com:bitcoindevkit/bdk 2020-12-16 10:48:10 +01:00
Steve Myers 148e8c6088 [docs] Add docs to the 'wallet' module 2020-12-15 15:12:32 -08:00
Evgenii P 09730c0898 Take ID into account in SignersContainerKey's PartialEq impl 2020-12-15 22:40:07 +07:00
Alekos Filini 6d9472793c Merge commit 'refs/pull/228/head' of github.com:bitcoindevkit/bdk 2020-12-15 14:33:59 +01:00
Alekos Filini eadf50042c [wallet] Add tests for check_nsequence_rbf and check_nlocktime 2020-12-15 12:01:44 +01:00
Alekos Filini 322122afc8 [wallet] Set the correct nSequence when RBF and OP_CSV are used 
This commit also fixes the timelock comparing logic in the policy module, since
the rules are different for absolute (OP_CLTV) and relative (OP_CSV) timelocks.

Fixes #215
 2020-12-15 12:01:41 +01:00
Evgenii P 5315c3ef25 rustfmt 2020-12-15 11:36:26 +07:00
Evgenii P c58236fcd7 Fix SignersContainer::find to filter out incorrect IDs 2020-12-15 11:36:26 +07:00
Evgenii P 2658a9b05a Fix SignersContainerKey PartialOrd to respect the ID 2020-12-15 11:36:26 +07:00
Evgenii P c075183a7b Revert replacing BTreeMap to HashMap in SingersContainer 2020-12-15 11:35:34 +07:00
LLFourn 9b31ae9153 Fix doc comment fallout from s/script type/keychain 2020-12-15 08:39:19 +11:00
Alekos Filini 1713d621d4 Rename ScriptType to KeychainKind 
This avoids confusion with the "type of script".
 2020-12-14 17:14:24 +01:00
Alekos Filini 0ef0b45745 Merge commit 'refs/pull/224/head' of github.com:bitcoindevkit/bdk 2020-12-14 11:18:51 +01:00
Evgenii P 351b656a82 Use unstable sort by key for performance 2020-12-14 16:27:54 +07:00
Evgenii P 95af38a01d rustfmt 2020-12-14 01:03:14 +07:00
Evgenii P 3ceaa33de0 Add unit tests for SignersContainer 2020-12-14 01:03:14 +07:00
Evgenii P 5d190aa87d Remove debug output 2020-12-14 01:02:48 +07:00
Evgenii P 20e0a4d421 Replace BTreeMap with a HashMap 2020-12-13 18:37:27 +07:00
Alekos Filini 9bafdfe2d4 [docs] Various fixes to the docs 2020-12-11 11:16:38 +01:00
Steve Myers 602ae3d63a Add TODOs for missing_docs 2020-12-07 18:25:16 -08:00
Alekos Filini eef59e463d Merge commit 'refs/pull/210/head' of github.com:bitcoindevkit/bdk 2020-12-07 11:21:21 +01:00
LLFourn 8dcb75dfa4 Replace UTXO::is_internal with script_type 
This means less conversion and logic mapping from bool to ScriptType and
back again.
 2020-12-04 10:46:25 +11:00
Steve Myers 2e7f98a371 Fix docs 2020-12-02 16:57:59 -08:00
Alekos Filini 5d352ecb63 [wallet] Add tests for TxBuilder::add_global_xpubs() 2020-12-01 16:43:40 +01:00
Alekos Filini ebfe5db0c3 [wallet] Add a flag to fill-in PSBT_GLOBAL_XPUB 2020-12-01 16:43:38 +01:00