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

rpcclient: sendrawtransaction is broken for Bitcoin Core #1646

Closed
torkelrogstad opened this issue Oct 6, 2020 · 1 comment
Closed

rpcclient: sendrawtransaction is broken for Bitcoin Core #1646

torkelrogstad opened this issue Oct 6, 2020 · 1 comment

Comments

@torkelrogstad
Copy link
Contributor

It is not possible to use sendrawtransaction with the current RPC client (0.21) and Bitcoin Core (0.20, believe this applies to 0.19) as well. This regression was introduced in #1484.

The underlying issue is that Bitcoin Core expects two arguments to the RPC call, and the current implementation leads to three.

I tried to fix this on my side by adding two different structs, one for pre- and post 0.19. I then ran into a blocker, where we can only register one struct per command.

It seems like this was attempted fixed in #1415. I'd say that this is a pretty high priority issue. I can work around it for now by using a fork, but that's not ideal.

@onyb
Copy link
Contributor

onyb commented Nov 16, 2020

Fixed in #1659.

@onyb onyb closed this as completed Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants