NodeID is not emitted in ValidatorManager Events

[0xJohnnyGault]

In ValidatorManager.sol, NodeID is defined as bytes (a dynamically-sized type). I understand that this is probably done for future-proofing. However, when events are emitted like:

  event ValidationPeriodCreated(
    bytes32 indexed validationID,
    bytes indexed nodeID,
    bytes32 indexed registerValidationMessageID,
    uint256 weight,
    uint64 registrationExpiry
  );

Its misleading, because the actual NodeID is not emitted, as Solidity will keccak the bytes to get a 32-byte hash to put in the event. So at a minimum I would suggest renaming it to NodeIDHash which would make it clear its not the actual nodeid.

Alternatively, I think if indexed is removed, the full value would be placed in the event log.