Do not initialize log level to ensure each case is handled explicitly

ChainSafe · Feb 15, 2024 · 4408a92 · 4408a92
1 parent 68c6172
commit 4408a92
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/packages/beacon-node/src/network/processor/gossipHandlers.ts b/packages/beacon-node/src/network/processor/gossipHandlers.ts
@@ -255,8 +255,8 @@ function getDefaultHandlers(modules: ValidatorFnsModules, options: GossipHandler
         chain.seenGossipBlockInput.prune();
       })
       .catch((e) => {
-        // Any unexpected error
-        let logLevel = LogLevel.error;
+        // Adjust verbosity based on error type
+        let logLevel: LogLevel;
 
         if (e instanceof BlockError) {
           switch (e.type.code) {
@@ -287,6 +287,9 @@ function getDefaultHandlers(modules: ValidatorFnsModules, options: GossipHandler
               // Misbehaving peer, user cannot be expected to do something about this
               logLevel = LogLevel.warn;
           }
+        } else {
+          // Any unexpected error
+          logLevel = LogLevel.error;
         }
         metrics?.gossipBlock.processBlockErrors.inc({error: e instanceof BlockError ? e.type.code : "NOT_BLOCK_ERROR"});
         logger[logLevel]("Error receiving block", {slot: signedBlock.message.slot, peer: peerIdStr}, e as Error);