-
Notifications
You must be signed in to change notification settings - Fork 86
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
client: add support for dcr spv wallets #788
Merged
Merged
Commits on Nov 8, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 0451243 - Browse repository at this point
Copy the full SHA 0451243View commit details -
client/asset/dcr: add basic support for spv wallets
- Don't error for missing dcrdjsonrpcapi, instead set wallet.spvMode=true. - Replace getblockchaininfo with syncstatus to determine sync status.
Configuration menu - View commit details
-
Copy full SHA for 8fa63e5 - Browse repository at this point
Copy the full SHA 8fa63e5View commit details -
Configuration menu - View commit details
-
Copy full SHA for c3d9007 - Browse repository at this point
Copy the full SHA c3d9007View commit details -
Configuration menu - View commit details
-
Copy full SHA for af46a25 - Browse repository at this point
Copy the full SHA af46a25View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2355326 - Browse repository at this point
Copy the full SHA 2355326View commit details -
multi: re-audit contracts after confirmation, before acting
There is a very slight possibility that contract outputs that pass audit while in mempool or in some cases before they are broadcasted (rawtx audit) may differ from the output later observed in a block for the same coin. This risk is higher for spv wallets that will mostly only perform audits on rawtxs before broadcasting the txs, without a guarantee that the tx is accepted to the mempool. A malicious actor could broadcast a different tx with same hash (theoretically possible) but with a different output at the expected vout index. There is risk of funds if clients only later check that the hash for the earlier-audited tx is found in a block and proceed to send their counter swap or expose their contract secret via a redemption. This commit aims to mitigate that risk by repeating contract audits after the initial tx hash is observed on the blockchain, ensuring that the tx now observed on the blockchain is as desired.
Configuration menu - View commit details
-
Copy full SHA for 4f5a5e2 - Browse repository at this point
Copy the full SHA 4f5a5e2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3c11408 - Browse repository at this point
Copy the full SHA 3c11408View commit details -
client/asset/dcr: remove added calls to getrawtransaction
The getrawtransaction rpc requires txindex to be enabled on full nodes but clients may run full nodes without enabling txindex as it is not a requirement. Use gettransaction where possible instead of getrawtransaction to avoid errors when clients use full nodes without txindex enabled.
Configuration menu - View commit details
-
Copy full SHA for 206f27d - Browse repository at this point
Copy the full SHA 206f27dView commit details -
Configuration menu - View commit details
-
Copy full SHA for b0749cf - Browse repository at this point
Copy the full SHA b0749cfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7e211c8 - Browse repository at this point
Copy the full SHA 7e211c8View commit details -
fix dcr harness spv wallets startup issues fix SwapConfirmations error for wallet contracts fix output spent check bug and repaired log messages
Configuration menu - View commit details
-
Copy full SHA for ad2e26c - Browse repository at this point
Copy the full SHA ad2e26cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 742432e - Browse repository at this point
Copy the full SHA 742432eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 19fd34a - Browse repository at this point
Copy the full SHA 19fd34aView commit details -
Configuration menu - View commit details
-
Copy full SHA for e569f68 - Browse repository at this point
Copy the full SHA e569f68View commit details -
Configuration menu - View commit details
-
Copy full SHA for eb24720 - Browse repository at this point
Copy the full SHA eb24720View commit details -
Configuration menu - View commit details
-
Copy full SHA for 785973a - Browse repository at this point
Copy the full SHA 785973aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5a53e29 - Browse repository at this point
Copy the full SHA 5a53e29View commit details -
Configuration menu - View commit details
-
Copy full SHA for ab0d0bd - Browse repository at this point
Copy the full SHA ab0d0bdView commit details -
Configuration menu - View commit details
-
Copy full SHA for d798528 - Browse repository at this point
Copy the full SHA d798528View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f884a5 - Browse repository at this point
Copy the full SHA 5f884a5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 73ca58c - Browse repository at this point
Copy the full SHA 73ca58cView commit details -
Configuration menu - View commit details
-
Copy full SHA for b20b179 - Browse repository at this point
Copy the full SHA b20b179View commit details
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.