search

celestiaorg / celestia-node

Celestia Data Availability Nodes
Apache License 2.0
932 stars 926 forks source link

initial POC for grpc tls #3917

Closed jhawk28 closed 2 days ago

jhawk28 commented 2 weeks ago

Initial concept for gRPC tls support

a26nine commented 2 weeks ago

I was able to sync a light node with:

[Core]
  IP = "xxx.celestia-mainnet.quiknode.pro"
  RPCPort = "443"
  GRPCPort = "9090"
  EnableTLS = true

using this build.

codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 57.89474% with 8 lines in your changes missing coverage. Please review.

Project coverage is 45.42%. Comparing base (2469e7a) to head (d187ddd). Report is 376 commits behind head on main.

Files with missing lines Patch % Lines
nodebuilder/core/flags.go 55.55% 3 Missing and 1 partial :warning:
state/core_access.go 66.66% 1 Missing and 2 partials :warning:
nodebuilder/state/core.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3917 +/- ## ========================================== + Coverage 44.83% 45.42% +0.58% ========================================== Files 265 307 +42 Lines 14620 21849 +7229 ========================================== + Hits 6555 9924 +3369 - Misses 7313 10855 +3542 - Partials 752 1070 +318 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

vgonkivs commented 2 days ago

closed in favor of #3954