Skip to content
This repository has been archived by the owner on Jun 3, 2020. It is now read-only.

tendermint-rs: Initial JSONRPC over HTTP client + broadcast_tx_* endpoints #243

Merged
merged 4 commits into from
Apr 24, 2019

Conversation

tarcieri
Copy link
Contributor

@tarcieri tarcieri commented Apr 22, 2019

Adds a hyper-based HTTP client for making JSONRPC requests.

Uses a slightly old version of hyper (0.10) which does not yet depend on futures/tokio, the idea being updating to a fully async version should be possible in the future.

Adds an integration test which can be used against gaiad locally. This isn't run in CI (yet) but it is built in CI at least to ensure changes still allow it to compile.

Additionally implements wrappers for the /broadcast_tx_* endpoints for broadcasting transactions.

Adds a hyper-based HTTP client for making JSONRPC requests.

Uses a slightly old version of hyper (0.10) which does not yet depend on
futures/tokio, the idea being updating to a fully async version should
be possible in the future.

Adds an integration test which can be used against gaiad locally. This
isn't run in CI (yet) but it is built in CI at least to ensure changes
still allow it to compile.
This allows us to use POST requests and serde serialization when
creating requests, rather than trying to shove things into URIs.
Support for all methods of broadcasting transactions presently provided
by the Tendermint RPC API.

This implementation has been unit tested against the responses
documented at https://tendermint.com/rpc/ but does not yet have live
integration tests.
@tarcieri tarcieri changed the title tendermint-rs: Initial JSONRPC over HTTP client tendermint-rs: Initial JSONRPC over HTTP client + broadcast_tx_* endpoints Apr 24, 2019
@tarcieri tarcieri merged commit 77bb14d into master Apr 24, 2019
@tarcieri tarcieri deleted the tendermint-rs/rpc-client branch April 24, 2019 16:57
@tarcieri tarcieri mentioned this pull request Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants