[EIP-7594] Add new getBlobSidecars spec #524
Conversation
types/fulu/blob_sidecar.yaml
Outdated
| items: | ||
| $ref: "#/Deneb/BlobSidecar" | ||
| minItems: 0 | ||
| maxItems: 48 # placeholder, depending on the final value we chose and BPO schedule |
There was a problem hiding this comment.
Hmm I feel that we should just use a really high value here instead. Let's discuss this first.
| maxItems: 48 # placeholder, depending on the final value we chose and BPO schedule | |
| maxItems: 4096 # MAX_BLOB_COMMITMENTS_PER_BLOCK |
There was a problem hiding this comment.
+1 on setting it high enough that we don't ever need to change it anymore
There was a problem hiding this comment.
there was some discussion in #488 on this previously, I am generally in favor of using sane list limits but since blob limit increase will be effectively decoupled from (named) forks with EIP-7892 I also think it's best to use 4096 now
There was a problem hiding this comment.
seems like we have enough consensus here, updated
0x00101010
force-pushed
the
fulu-blobsidecar
branch
from
f954bd7 to
6b4153b
Compare
| KZGCommitmentInclusionProof | ||
| LMD | ||
| fulu | ||
| BPO |
There was a problem hiding this comment.
this can be removed, the word was previously in comment for maxItems but removed in 6b4153b
|
Currently, this beacon API returns blob sidecars, including:
It seems main users of the Proposal: What about creating a new beacon API endpoint |
|
Closing this PR in favor of #546 |
5 participants
No description provided.