Small devp2p Namespace Fixes (#3949)

* Small devp2p namespace addition fix

* Small spelling fix

* Fix per-message ETH protocol debugging

Author: holgerd77

packages/devp2p/src/protocol/protocol.ts

@@ -54,8 +54,8 @@ export abstract class Protocol {
   }
 
   private initMsgDebuggers(protocol: ProtocolType) {
-    const MESSAGE_NAMES = Object.values(this._messageCodes).filter(
-      (value) => typeof value === 'string',
+    const MESSAGE_NAMES = Object.keys(this._messageCodes).filter(
+      (key) => typeof key === 'string',
     ) as string[]
     for (const name of MESSAGE_NAMES) {
       this.msgDebuggers[name] = devp2pDebug.extend(protocol).extend(name)

packages/devp2p/src/util.ts

@@ -12,7 +12,9 @@ import { secp256k1 } from 'ethereum-cryptography/secp256k1.js'
 
 import type { EthStatusMsg } from './protocol/eth.ts'
 
-export const devp2pDebug = debug('devp2p:#')
+// Do not use :# here, no logging without sub namespace occurring and current code structure
+// otherwise creates loggers like `devp2p:#:eth`
+export const devp2pDebug = debug('devp2p')
 
 export function genPrivateKey(): Uint8Array {
   const privateKey = secp256k1.utils.randomPrivateKey()