bump news
to fix concurrency bug with >1MB frames
#4028
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When calling
newPayload
on a >1MB payload (can happen post-merge),news
splits up that payload into 1MB chunks. The chunks are each sentindividually, though, with
await
in-between. This means that when wesend concurrent
forkChoiceUpdated
calls, that those may end up gettingin-between the
newPayload
chunks, leading to invalid data being sent.The EL then returns an error message with a
null
id
entry (as itcould not read the request
id
due to the mangling) and disconnects.A PR has been submitted to fix this in
news
, and merged intostatus
branch early as this fix is critical for reliable post-merge operation:
Tormund/news#22