Skip to content

Conversation

@lexnv
Copy link

@lexnv lexnv commented Jun 3, 2025

This PR ensures that the voter logic stops when the global_in stream is terminated.

While at it, have replaced the log dependency with tracing

@lexnv lexnv self-assigned this Jun 3, 2025
@lexnv lexnv changed the title voter: Stop the voter logic on terminated global inbound stream voter: Stop the voter logic on terminated global input stream Jun 3, 2025
warn!(
target: LOG_TARGET,
?err,
"Possible case of massive equivocation: \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably don't want this log line to trigger if it's a stream-closed error.

Copy link
Contributor

@andresilva andresilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall lgtm, just need to handle rob's comment (only log the equivocation statement if the error is NotDescendent)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants