ChainSafe / lodestar

🌟 TypeScript Implementation of Ethereum Consensus
https://lodestar.chainsafe.io
Apache License 2.0
1.15k stars 283 forks source link

chore: review node routes #6844

Closed nflaig closed 3 months ago

nflaig commented 4 months ago

Motivation

Pulled out of https://github.com/ChainSafe/lodestar/pull/6749 to discuss changes separately and for easier review.

Description

Just cleanup, but might wanna discuss how it could be possible to support ssz response for getNetworkIdentity. I know one blocker was moving the ENR functions into it's on package but I still see issues, as there is no JSON mapping for enr, peerid, etc.