-
Notifications
You must be signed in to change notification settings - Fork 79
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
feat: core gRPC services #842
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #842 +/- ##
==========================================
- Coverage 71.48% 68.28% -3.20%
==========================================
Files 124 129 +5
Lines 14979 15856 +877
==========================================
+ Hits 10707 10827 +120
- Misses 4272 5029 +757
☔ View full report in Codecov by Sentry. |
cf07ddc
to
67cd50b
Compare
Also, can you add a unclog as well? |
I'd like to circle in @keppel as well. It would be awesome to have your input and see if this aligns with what you've been looking for earlier. |
40854ae
to
7b9f375
Compare
Co-authored-by: Philippe Laferrière <plafer@protonmail.com> Signed-off-by: Rano | Ranadeep <ranadip.bswas@gmail.com>
will be fixed in separate PR
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.
🎉
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.
Thanks @rnbguy for taking care of this PR 🙏🏻
* add grpc service dependencies * client service implementation * connection service implementation * channel service implementation * finish grpc service implementations * QueryContext under grpc feature * fix no-default-features lints * use domain types * return query block height * happy lint * immut router in validate * client and connection param service * implement upgrade service * fix upgrade context * rename struct fields and generics * remove UpgradeValidationContext trait bound * cargo fmt and clippy * add ProvableContext * add proofs to client service * add proofs to connection service * add proofs to channel service * remove redundant trait method * refactor ProvableContext trait bounds * refactor service structs * add service doc * add context docs * from impl for tonic status * remove redundant map_errs * fix cargo doc lint * remove irrelevant comments * handle empty sequence list * a general variant of PackerError * polish comments and logging * remove grpc from default features * mv grpc feature gated code under service * mv files * update mod files * update prelude * happy cargo doc * happy cargo fmt * support latest_height flag * rm redundant trait method * comment on grpc feature * todo comment for pagination * update client_status query * rm redundant trait method * add changelog entry * nitpick * rm allowed_clients * update client params query * update error message Co-authored-by: Philippe Laferrière <plafer@protonmail.com> Signed-off-by: Rano | Ranadeep <ranadip.bswas@gmail.com> * revert immut ref on validate will be fixed in separate PR * rm tracing use * document thread-safe params * point changelog entry to issue * rm FromStr from prelude * doc strings with correct explanations * rm reexports --------- Signed-off-by: Rano | Ranadeep <ranadip.bswas@gmail.com> Co-authored-by: Philippe Laferrière <plafer@protonmail.com>
Closes: #686
Description
This PR implements IBC core gRPC services for
ibc-rs
using context trait definitions.This implementation aimed to leverage
ValidationContext
only. But some extensions were needed. SoQueryContext
is created extending the originalValidationContext
.Projects using
ibc-rs
can implementValidationContext
andQueryContext
and have IBC core gRPC services for free.A feature flag
grpc
is introduced to (en/dis)able this feature accordingly.PR author checklist:
QueryContext
impl for MockContextgrpc
from default feature setunclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.