From: Matt Corallo <git@bluematt.me>
Date: Sat, 15 Feb 2020 03:51:09 +0000 (-0500)
Subject: Don't send padding bytes for pings
X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=cc6dd456faa878492f8f235b5d1770ca108271a6;p=rust-lightning

Don't send padding bytes for pings

We (somewhat needlessly) send an extra 64 bytes with every ping.
While this isn't exactly a big deal, I saw it and figured we
shouldn't waste bytes on the wire given some of the constraints of
potential users.
---

diff --git a/lightning/src/ln/peer_handler.rs b/lightning/src/ln/peer_handler.rs
index 692e6769a..1814f0af8 100644
--- a/lightning/src/ln/peer_handler.rs
+++ b/lightning/src/ln/peer_handler.rs
@@ -1182,7 +1182,7 @@ impl<Descriptor: SocketDescriptor, CM: Deref, L: Deref> PeerManager<Descriptor,
 
 				let ping = msgs::Ping {
 					ponglen: 0,
-					byteslen: 64,
+					byteslen: 0,
 				};
 				peer.pending_outbound_buffer.push_back(peer.channel_encryptor.encrypt_message(&encode_msg!(&ping)));