log the public key of the validator claiming blocks too frequently

Signed-off-by: Benoit Razet <benoit.razet@pokitdok.com>
hyperledger-archives · Apr 20, 2018 · 7b40f34 · 7b40f34
1 parent 8946d7b
commit 7b40f34
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/core/sawtooth_poet/poet_consensus/poet_block_publisher.py b/core/sawtooth_poet/poet_consensus/poet_block_publisher.py
@@ -536,9 +536,11 @@ def initialize_block(self, block_header):
                 block_cache=self._block_cache,
                 poet_enclave_module=poet_enclave_module):
             LOGGER.info(
-                'Reject building on block %s: Validator is claiming blocks '
+                'Reject building on block %s: '
+                'Validator (signing public key: %s) is claiming blocks '
                 'too frequently.',
-                block_header.previous_block_id[:8])
+                block_header.previous_block_id[:8],
+                block_header.signer_public_key)
             return False
 
         # At this point, we know that if we are able to claim the block we are

diff --git a/core/tests/test_consensus/test_poet_block_publisher.py b/core/tests/test_consensus/test_poet_block_publisher.py
@@ -621,7 +621,7 @@ def test_z_policy(
             # function fails for the reason we expect.
 
             (message, *_), _ = mock_logger.info.call_args
-            self.assertTrue('Validator is claiming blocks too '
+            self.assertTrue('is claiming blocks too '
                             'frequently' in message)
 
     @mock.patch('sawtooth_poet.poet_consensus.poet_block_publisher.'