lodestar icon indicating copy to clipboard operation
lodestar copied to clipboard

Remove v1 attestation apis

Open nflaig opened this issue 11 months ago • 5 comments

The v1 attestation apis become effectively useless after electra hard fork since they only support the phase0 format. The v2 apis are fork-aware and backward compatible so it might be fine to just remove v1 soon after the hard fork but we likely need to wait until fulu for them to be removed from the beacon-api spec.

nflaig avatar Jan 30 '25 12:01 nflaig

Same with this, could probably tag as Fulu as we can remove it then.

philknows avatar Feb 03 '25 13:02 philknows

Same with this, could probably tag as Fulu as we can remove it then.

I am not sure how aggressive we wanna be at removing those, technically could do it in our next release right after Electra but might keep them around for another fork

nflaig avatar Feb 03 '25 13:02 nflaig

We don't need to be aggressive with it especially if we have more important matters to put our time to. But thanks for creating the issue to track removal though.

philknows avatar Feb 03 '25 21:02 philknows

we should just clean up caller side, we can't remove them from the beacon node as per spec can only be done after Fulu goes live

nflaig avatar Apr 10 '25 12:04 nflaig

Should do post-Fulu mainnet release

philknows avatar Aug 12 '25 15:08 philknows