Skip to content

Conversation

matthew-alexander-partior

This PR aims to simplify debugging validator set related issues

Problem Statement:
Given a quorum IBFT network, with a validator set that is a subset of the network

When a node that is not part of the validator set sends consensus message, the receiving node will log an error:
Failed to decode message from payload

In here, the receiving node does an ecrecover on the message sent, but the address recovered is not part of the validator set

Logging here not sufficient to determine:

  • Which node was sending the message that is not part of the validator set?
  • Who is in the validator set now?

Solution

  • Adds logs for validatorSet & sender node address when error occurs (on DEBUG mode)

* feat: SET-460 Add logging for validator set

* feat: SET-460 refactor code to make it 1 liner

* feat: SET-460 replace warn -> debug logs
@CLAassistant
Copy link

CLAassistant commented Jul 30, 2024

CLA assistant check
All committers have signed the CLA.

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.

2 participants