Skip to content
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

Adds native segwit script types #10

Open
wants to merge 40 commits into
base: master
Choose a base branch
from

Conversation

mariorz
Copy link

@mariorz mariorz commented Aug 30, 2017

BtcdClient.decodeRawTransaction raises an IllegalArgumentException on parsing native segwit script types: witness_v0_keyhash, witness_v0_scripthash

e.g.

java.lang.IllegalArgumentException: Error #1001010: Expected the argument to be a valid 'bitcoind' script type, but was invalid/unsupported instead. 2017-08-29 03:59:06.839431500 at com.neemre.btcdcli4j.core.domain.enums.ScriptTypes.forName(ScriptTypes.java:43)

This patch adds the missing enums to fix.

@mariorz mariorz closed this Sep 6, 2017
@mariorz mariorz reopened this Sep 6, 2017
vogelito and others added 23 commits October 14, 2017 21:27
-Support for bitcoin 0.16 segWit transactions (decodeRawTransaction has boolean flag isWitness)
-Change the inital sanity check from the depcreated getInfo() to getNetworkInfo()
-Support for locally installing test jars, see README.md for details.  Change pom.xml files to use SNAPSHOT version, as hopefully our jitpack can detail versioning.
* Remove estimateFee, new one is estimateSmartFee.  Update jdk depedency and lombok 
* updates from PR comments
Taproot - add support to p2tp script
Fix getRawTransaction with verbosity method in order to populate the …
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants