You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
so I think we can do this unless we change the ckzg validation interfaces
or we arc the entire sidecar, which would then be incompatible with the blobsbundlev1 type
Additional context
No response
The text was updated successfully, but these errors were encountered:
Component
consensus, eips, genesis
Describe the feature you would like
the current blob type is a fixedbytes alias allocated on the stack
alloy/crates/eips/src/eip4844/mod.rs
Lines 90 to 92 in 1ba17f2
this is expensive to clone and has issues with StackOverflow in serde.
ideally we want a this to behave like
Bytes
but enforces the length at all timeswe can solve this with a wrapper type
Blob(Bytes)
and only allow BYTES_PER_BLOBbut we need a way to transmute the blob to ckzg type so perhaps this should instead be
Arc<FixedBytes<BYTES_PER_BLOB>>
so this will be problematic
alloy/crates/eips/src/eip4844/sidecar.rs
Line 274 in 1ba17f2
so I think we can do this unless we change the ckzg validation interfaces
or we arc the entire sidecar, which would then be incompatible with the blobsbundlev1 type
Additional context
No response
The text was updated successfully, but these errors were encountered: