-
Notifications
You must be signed in to change notification settings - Fork 56
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
Update rust-jsonrpc
and switch to using minreq
#562
Conversation
rust-jsonrpc
and switch to using minreq
rust-jsonrpc
and switch to using minreq
ad9ddb7
to
0ad65d0
Compare
Alright so upstream was updated in apoelstra/rust-jsonrpc#103 apoelstra/rust-jsonrpc#102 to support everything we need. I updated this branch with a functional test exercising the request retry logic. |
Looks like nightly is broken.. |
Fixed the compilation issue on nightly (as per dtolnay/proc-macro2#356 (comment)) in #565. I'll rebase on top of it. |
rust-jsonrpc
and switch to using minreq
rust-jsonrpc
and switch to using minreq
Rebased, this is now ready for review. |
Instead of the hand-written HTTP implementation in the rust-jsonrpc library, use the minreq crate. It's a small, maintained, low dependency library actually focused on writing an HTTP client. This also reworks and better document the request retry logic.
ACK 43a0cf2 -- let's get it in early and try it as we go. |
This makes us take advantage of a more robust, but still lightweight, HTTP implementation.
This PR also cleanups our error handling code and adds a functional test checking we do retry request on transient bitcoind failures.