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
The current QBFT implementation is buffering ALL messages.
A valid message may however be unjust.
This is either due to a bug in the code, or due to Byzantine behaviour
To err on the side of caution, it should be assumed the be Byzantine, and those messages should not be buffered.
Proposed solution
Move buffering of messages after classification.
Note that classification assumes the message is already in the buffer, create a local var all []Msg slice that includes the latest message (without modifying the buffer itself).
The text was updated successfully, but these errors were encountered:
The current QBFT implementation is buffering ALL messages.
A valid message may however be unjust.
This is either due to a bug in the code, or due to Byzantine behaviour
To err on the side of caution, it should be assumed the be Byzantine, and those messages should not be buffered.
category: refactor
ticket: #525
Problem to be solved
The current QBFT implementation is buffering ALL messages.
A valid message may however be unjust.
This is either due to a bug in the code, or due to Byzantine behaviour
To err on the side of caution, it should be assumed the be Byzantine, and those messages should not be buffered.
Proposed solution
Move buffering of messages after classification.
Note that classification assumes the message is already in the buffer, create a local
var all []Msg
slice that includes the latest message (without modifying the buffer itself).The text was updated successfully, but these errors were encountered: