-
Couldn't load subscription status.
- Fork 128
core/validatorapi: add eth2wrap logic to proxy #4042
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
base: main
Are you sure you want to change the base?
Conversation
4e4e938 to
973b9e4
Compare
9c4a472 to
4a4e409
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #4042 +/- ##
==========================================
+ Coverage 53.80% 53.82% +0.01%
==========================================
Files 242 242
Lines 39425 39594 +169
==========================================
+ Hits 21214 21312 +98
- Misses 15967 16033 +66
- Partials 2244 2249 +5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
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.
Looks like the build is broken



Proxied requests from the VC to the BN didn't have our
eth2wraplogic (such as sending to multiple BNs or retrying to fallback nodes) neither did they send our additional headers fromCHARON_BEACON_NODE_HEADERS.This changes our proxy to use the
eth2wrapclient to proxy to (possibly) multiple addresses instead of directly proxing to a single address.category: feature
ticket: none