bithyve / bitcoin-keeper

Secure today, plan for tomorrow. BIP 85 hot wallets, buy bitcoin, tor privacy.
https://bitcoinkeeper.app
MIT License
32 stars 11 forks source link

Cached Tx: Transaction priority lock #4805

Open Parsh opened 1 week ago

Parsh commented 1 week ago

Task Description Tx priority needs to be locked for cached transactions, as once inputs are selected based on priority(each priority can have a different set of inputs based on the fee being paid - we perform 3 coinselects), and are signed by one of the signers, all other signers will need to sign the same set of inputs otherwise the transaction will have incorrect signatures. Hence switching the transaction priority should not be allowed for a cached transaction.

Note: We might want to also cache the original priority for the transaction when it was initiated before it was cached for the first time.