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

feat!(core): remove dependency of RPC in favour of gRPC #3998

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

vgonkivs
Copy link
Member

The PR adds a gRPC streaming for the block sync for Bridge nodes and enables a secure connection over auth token

@vgonkivs vgonkivs added area:core_and_app Relationship with Core node and Celestia-App kind:break! Attached to breaking PRs kind:misc Attached to miscellaneous PRs labels Dec 11, 2024
@vgonkivs vgonkivs self-assigned this Dec 11, 2024
@vgonkivs vgonkivs marked this pull request as draft December 11, 2024 11:29
@Wondertan
Copy link
Member

Wondertan commented Dec 13, 2024

Reminder to merge this as independent commits

@vgonkivs vgonkivs force-pushed the feature_branch_grpc branch from 15e66a6 to 5326d1b Compare January 15, 2025 11:14
@vgonkivs vgonkivs requested a review from rach-id January 16, 2025 14:14
@vgonkivs vgonkivs marked this pull request as ready for review January 16, 2025 14:14
rach-id
rach-id previously approved these changes Jan 16, 2025
Copy link
Member

@rach-id rach-id left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@vgonkivs vgonkivs force-pushed the feature_branch_grpc branch from 1742249 to 5cdb112 Compare January 20, 2025 14:04
@renaynay renaynay merged commit 8cb979b into main Jan 21, 2025
27 of 30 checks passed
@renaynay renaynay deleted the feature_branch_grpc branch January 21, 2025 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core_and_app Relationship with Core node and Celestia-App kind:break! Attached to breaking PRs kind:misc Attached to miscellaneous PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants