12 Commits (6e7e39eea4f1149cd6490a0204c6545a86c3c14f)

Author	SHA1	Message	Date
Byron Hambly	6e7e39eea4	build(deps): upgrade bdk, ecdsa_fun, sigma_fun, sha2, bitcoin-harness, serde ... - bdk to 0.22.0 #1126 - ecdsa_fun to 7c3d592 #1127 - sigma_fun to 7c3d592 #1128 - sha2 to 0.10.2 #948 - serde to 1.0.144 #1115 - bitcoin-harness to bff9a64 Revert "ci: specify previous dprint version until fixed" This reverts commit 11eb1737ce.	2 years ago
Thomas Eizinger	4de8b0ebc4	Upgrade bitcoin and bdk to latest version ... The latest version allows us to access `miniscript` via `bdk` which removes the need for declaring it as an extra dependency.	3 years ago
Philipp Hoenisch	dc6ab0fa52	Ensure that constant weights do not go out of sync with code.	3 years ago
Philipp Hoenisch	ee90c228b4	Dynamically calculate fees using electrum's estimate_fee. ... Electrum has an estimate-fee feature which takes as input the block you want a tx to be included. The result is a recommendation of BTC/vbyte. Using this recommendation and the knowledge about the size of our transactions we compute an appropriate fee. The size of the transactions were taken from real transactions as published on bitcoin testnet. Note: in reality these sizes might fluctuate a bit but not for much.	3 years ago
Philipp Hoenisch	1012e39527	Dynamically chose fee for TxRefund and TxPunish. ... Alice chooses the fee for TxPunish because she is the one that cares. Bob chooses the fee for TxRefund because he is the one that cares. Note must be taken here because if the fee is too low (e.g. < min tx fee) then she might not be able to publish TxRedeem at all.	3 years ago
Thomas Eizinger	273cf15631	Introduce `Watchable` abstraction for Bitcoin wallet ... We have a repeated pattern where we construct one of our Tx{Cancel,Redeem,Punish,Refund,Lock} transactions and wait until the status of this transaction changes. We can make this more ergonomic by creating and implementing a `Watchable` trait that gives access to the TxId and relevant script for this transaction. This allows us to remove a parameter from the `watch_until_status` function. Additionally, there is a 2nd pattern: "Completing" one of these transaction and waiting until they are confirmed with the configured number of blocks for finality. We can make this more ergonomic by returning a future from `broadcast` that callers can await in case they want to wait for the broadcasted transaction to reach finality.	3 years ago
Thomas Eizinger	6beb732e35	Eliminate `build_bitcoin_punish_transaction` ... We reduce indirection by constructing TxPunish directly based off `State3` and make the type itself more powerful by moving the logic of completing it with a signature onto it.	3 years ago
Thomas Eizinger	6d9b21cb47	Change `imports_granularity` to module ... This reduces the overall amount of LoC that imports take up in our codebase by almost 100. It also makes merge-conflicts less likely because there is less grouping together of imports that may lead to layout changes which in turn can cause merge conflicts.	3 years ago
Thomas Eizinger	6c38d66864	Remove `Tx` arguments from `add_signatures` functions ... The only reason we need this argument is because we need to access the output descriptor. We can save that one ahead of time at when we construct the type.	3 years ago
Thomas Eizinger	84bc2c82b7	Upgrade to bdk 4.0 ... To achieve this we also: - upgrade rust-bitcoin to 0.26 - upgrade bitcoin-harness to latest version (which also depends bitcoin 0.26) - upgrade to latest edcsa-fun - replace cross_curve_dleq proof with sigma_fun (to avoid an upgrade dance over there)	3 years ago
Franck Royer	b8a84aa34b	Avoid possible mix up between timelocks ... Introduce new type to ensure no mix up happens when ordering the fields in function calls.	3 years ago
Franck Royer	9a321a4f09	Move transaction types in dedicated modules ... To allow the related timelock to be defined with the transaction that uses it. This will allow the access to the timelock's struct inner field with defining `From` impl.	3 years ago