[bisq-network/bisq] [WIP] Segwit for the trade protocol (#4612)
Oscar Guindzberg
notifications at github.com
Fri Oct 9 22:37:13 UTC 2020
<!--
- make yourself familiar with the CONTRIBUTING.md if you have not already (https://github.com/bisq-network/bisq/blob/master/CONTRIBUTING.md)
- make sure you follow our [coding style guidelines][https://github.com/bisq-network/style/issues)
- pick a descriptive title
- provide some meaningful PR description below
- create the PR
- in case you receive a "Change request" and/or a NACK, please react within 30 days. If not, we will close your PR and it can not be up for compensation.
- After addressing the change request, __please re-request a review!__ Otherwise we might miss your PR as we tend to only look at pull requests tagged with a "review required".
-->
Fixes #replaceWithIssueNr, fixes #replaceWithIssueNr
Your PR description here.
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/4612
-- Commit Summary --
* Segwit for the trade protocol start
* Use local bitcoinj
* Add constant SEGWIT_TRADE_PROTOCOL_ACTIVE
* Add Capability.TRADE_PROTOCOL_SEGWIT
* Trade protocol documentation
* WIP Review TradeWalletService
* Disable timeouts for local debugging
* Use segwit addresses - to be reviewed if backwards compatibility needed
* BtcWalletService: make segwit a variable
* Revert "Validate AddressEntry.segwit"
* Revert "Make it clear segwit is not used for the trade protocol yet."
* Revert "Construct dummy outputs with LegacyAddress"
* Revert "Switch back to LegacyAddress for fee estimation"
* Serialize bitcoin tx with segwit data
* Use p2wsh
* Use p2wsh
* Emergency publish payout tx: select segwit from the UI
* Create the scriptCode the right way
* Accept empty scriptSig in P2WPKH
* Replace TradeWalletService.getInput() by reading output value
* Use P2WPKH for trade protocol only is activated
* Revert to bitcoinSerialize(false) in RawTransactionInput
* Remove unused imports
* TransactionWitness.redeemP2WSH() now receives a Script
-- File Changes --
M build.gradle (7)
M common/src/main/java/bisq/common/app/Capability.java (3)
M common/src/main/java/bisq/common/config/Config.java (2)
M core/src/main/java/bisq/core/app/WalletAppSetup.java (2)
M core/src/main/java/bisq/core/btc/model/AddressEntry.java (4)
M core/src/main/java/bisq/core/btc/wallet/BtcWalletService.java (14)
M core/src/main/java/bisq/core/btc/wallet/TradeWalletService.java (197)
M core/src/main/java/bisq/core/btc/wallet/WalletService.java (7)
M core/src/main/java/bisq/core/trade/protocol/tasks/buyer/BuyerSignsDelayedPayoutTx.java (8)
M core/src/main/java/bisq/core/trade/protocol/tasks/seller/SellerSignsDelayedPayoutTx.java (7)
M desktop/src/main/java/bisq/desktop/main/overlays/windows/ManualPayoutTxWindow.java (8)
A docs/trade-protocol.md (81)
M gradle/witness/gradle-witness.gradle (2)
M p2p/src/main/java/bisq/network/p2p/network/Connection.java (2)
M p2p/src/main/java/bisq/network/p2p/network/NetworkNode.java (2)
A segwitForTheTradeProtocl.txt (1)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/4612.patch
https://github.com/bisq-network/bisq/pull/4612.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4612
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20201009/ecb17424/attachment.html>
More information about the bisq-github
mailing list