refactor(state): Remove need for core RPC conn and rely only on gRPC for state reads

[renaynay]

Resolves #3357

Blocked on patch release from app including celestiaorg/celestia-app#3424

Corresponding docs PR celestiaorg/docs#1558

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 73.68421% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 44.56%. Comparing base (2469e7a) to head (cfa8fa4).
Report is 54 commits behind head on main.

Files	Patch %	Lines
state/core_access.go	77.77%	2 Missing and 2 partials ⚠️
nodebuilder/state/core.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3359      +/-   ##
==========================================
- Coverage   44.83%   44.56%   -0.28%     
==========================================
  Files         265      272       +7     
  Lines       14620    15244     +624     
==========================================
+ Hits         6555     6793     +238     
- Misses       7313     7666     +353     
- Partials      752      785      +33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[liamsi]

Is it in scope of this Pr to ensure TLS encryption works for gRPC? And for bridge nodes TLS for RPC to core? If not I assume this will be tackled shortly after?

[Wondertan]

Is it in scope of this Pr to ensure TLS encryption works for gRPC? And for bridge nodes TLS for RPC to core? If not I assume this will be tackled shortly after?

It's good to have if for privacy reasons so that no MITM can see the addresses and balances you request. Worth having an issue

[Wondertan]

LGTM otherwise

[liamsi]

It's good to have if for privacy reasons so that no MITM can see the addresses and balances you request. Worth having an issue

There was an over 10 months old issue and it was closed... Rene opened a new one #3377

[Wondertan]

#2439 (comment)

🤔

I guess Ramin misunderstood the relation of that issue because one does not replace the other one.

[ramin]

#2439 (comment)

"Make secured connection possible"