fix: connection monitor compatible with other ping implementations (#2671)

Update the number of bytes sent by the connection manager so that it is compatible with other `ping` implementations

https://github.com/libp2p/go-libp2p/blob/master/p2p/protocol/ping/ping.go#L23
https://github.com/libp2p/rust-libp2p/blob/master/protocols/ping/src/protocol.rs#L48
https://github.com/libp2p/js-libp2p/blob/main/packages/protocol-ping/src/constants.ts#L2

Author: dozyio

packages/libp2p/src/connection-monitor.ts

@@ -1,3 +1,4 @@
+import { randomBytes } from '@libp2p/crypto'
 import { serviceCapabilities } from '@libp2p/interface'
 import { AdaptiveTimeout } from '@libp2p/utils/adaptive-timeout'
 import { byteStream } from 'it-byte-stream'
@@ -9,6 +10,7 @@ const DEFAULT_PING_INTERVAL_MS = 10000
 const PROTOCOL_VERSION = '1.0.0'
 const PROTOCOL_NAME = 'ping'
 const PROTOCOL_PREFIX = 'ipfs'
+const PING_LENGTH = 32
 
 export interface ConnectionMonitorInit {
   /**
@@ -103,10 +105,10 @@ export class ConnectionMonitor implements Startable {
             start = Date.now()
 
             await Promise.all([
-              bs.write(new Uint8Array(1), {
+              bs.write(randomBytes(PING_LENGTH), {
                 signal
               }),
-              bs.read(1, {
+              bs.read(PING_LENGTH, {
                 signal
               })
             ])