Rootul P
Rootul P
I think this is actually enforced because if a block proposer rearranges blobs in a namespace then that will result in a data root mismatch in process proposal. So the...
If we wanted the default layout ordering rules to only be defaults and allow other valid layouts, I think block proposers would need a mechanism to pass information about the...
Note we already have an ante handler that rejects txs where the total blob size is too large. It's possible that ante handler doesn't catch edge cases near the total...
`getTotal` is only used [here](https://github.com/celestiaorg/celestia-app/blob/2d261a2a0f7f627044342929b534a16b762c47e0/x/blob/ante/max_total_blob_size_ante.go#L33) where we want the number of total bytes in the blob sizes. > Also I would consider of adding an extra share or two as...
🤦♂️ you're right, good catch!
This issue may be partially mitigated by the fix in #3082 which will be released in the next major release.
TBH I still think this is open. #3082 improves the ante handler to reject PFBs that are guaranteed to not fit in a data square. But I think the DoS...
> what do we need to do to be confident that this isn't occuring? One more idea to mitigate the DoS: modify `BroadcastTx` from blocking to non-blocking > do we...
One idea that came up in discussion with @cmwaters is that we can roll this change out slowly by: 1. Adding a new msg type that doesn't include these fields...
kk if super optional, let's close as won't do for now. Please reopen if there's a compelling reason to do this