-
Notifications
You must be signed in to change notification settings - Fork 216
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
Add TLS support for the Light Client #842
Conversation
Signed-off-by: Thane Thomson <connect@thanethomson.com>
This commit adds pagination to the `validators` method on the `Client` trait (BREAKING). Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
… string (like page numbers/per page counts) Signed-off-by: Thane Thomson <connect@thanethomson.com>
…from strings first Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Codecov Report
@@ Coverage Diff @@
## thane/831-validators-hash #842 +/- ##
===========================================================
Coverage ? 28.8%
===========================================================
Files ? 196
Lines ? 10886
Branches ? 4495
===========================================================
Hits ? 3136
Misses ? 4773
Partials ? 2977 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Just left a single request.
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
* Remove unused file Signed-off-by: Thane Thomson <connect@thanethomson.com> * Refactor validators RPC endpoint interface This commit adds pagination to the `validators` method on the `Client` trait (BREAKING). Signed-off-by: Thane Thomson <connect@thanethomson.com> * Ensure "total" response field is a string Signed-off-by: Thane Thomson <connect@thanethomson.com> * Add serializer for optional types that need to be converted to/from a string (like page numbers/per page counts) Signed-off-by: Thane Thomson <connect@thanethomson.com> * Refactor to ensure page numbers and per-page values are converted to/from strings first Signed-off-by: Thane Thomson <connect@thanethomson.com> * Convert tcp:// scheme to http:// for RPC addresses Signed-off-by: Thane Thomson <connect@thanethomson.com> * Add Light Client support for RPC URLs instead of net::Address Signed-off-by: Thane Thomson <connect@thanethomson.com> * Revert 14ad69f for now Signed-off-by: Thane Thomson <connect@thanethomson.com> * Revert f0c26f7 Signed-off-by: Thane Thomson <connect@thanethomson.com> * Add CHANGELOG Signed-off-by: Thane Thomson <connect@thanethomson.com> * Convert tcp:// scheme to http:// for RPC addresses Signed-off-by: Thane Thomson <connect@thanethomson.com> * Add Light Client support for RPC URLs instead of net::Address Signed-off-by: Thane Thomson <connect@thanethomson.com> * Comment not needed Signed-off-by: Thane Thomson <connect@thanethomson.com> * Expose rpc::Url type Signed-off-by: Thane Thomson <connect@thanethomson.com> * Update kvstore integration test to use rpc::Url Signed-off-by: Thane Thomson <connect@thanethomson.com> * Update CHANGELOG Signed-off-by: Thane Thomson <connect@thanethomson.com> * Remove debug output from height log Signed-off-by: Thane Thomson <connect@thanethomson.com> * Attach serialization directly to tendermint_rpc::Url Signed-off-by: Thane Thomson <connect@thanethomson.com> * Add some happy path tests for tendermint_rpc::Url parsing Signed-off-by: Thane Thomson <connect@thanethomson.com>
Closes #835.
It additionally introduces a little more info-level logging in the Light Node.
I've successfully manually tested the Light Node against two different Cosmos Hub validators, both accessed via HTTPS.