Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add sanity check for maximum block size #3171

Merged
merged 2 commits into from
Nov 17, 2020

Conversation

vyzo
Copy link
Contributor

@vyzo vyzo commented Aug 19, 2020

1MB - some space for pubsub metadata; we should not exceed it in practice, but this adds a sanity check to make sure we don't clog the receiver ends in pubsub.

@vyzo vyzo changed the title add sanitcy check for maximum block size add sanity check for maximum block size Aug 19, 2020
@magik6k magik6k added impact/consensus Impact: Consensus and removed status/deferred labels Sep 2, 2020
@arajasek arajasek added this to the 🚢Lotus v1.2.0 milestone Nov 8, 2020
@arajasek arajasek force-pushed the feat/max-block-size-sanity-check branch from 2ac3af7 to 8191654 Compare November 17, 2020 22:39
@arajasek arajasek changed the base branch from master to release/v1.2.0 November 17, 2020 22:39
@arajasek arajasek changed the base branch from master to release/v1.2.0 November 17, 2020 22:39
@magik6k magik6k merged commit 9328d98 into release/v1.2.0 Nov 17, 2020
@magik6k magik6k deleted the feat/max-block-size-sanity-check branch November 17, 2020 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/consensus Impact: Consensus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants