From ccd1dad71e776b304020049184212419719c8265 Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Fri, 1 Dec 2017 19:27:38 -0800 Subject: [PATCH] peer: set HtlcPoint to nil within logWireMessage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This new field was added as a recent modification to the spec, but the curve parameter within the attribute wasn’t set to nil. As a result this would result in a large degree of spam within the logs when set to trace mode. This commit fixes this issue by setting it to nil along with all the other pub keys within messages. --- peer.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/peer.go b/peer.go index bbf2dd7a1..6df43af83 100644 --- a/peer.go +++ b/peer.go @@ -947,12 +947,14 @@ func (p *peer) logWireMessage(msg lnwire.Message, read bool) { m.RevocationPoint.Curve = nil m.PaymentPoint.Curve = nil m.DelayedPaymentPoint.Curve = nil + m.HtlcPoint.Curve = nil m.FirstCommitmentPoint.Curve = nil case *lnwire.OpenChannel: m.FundingKey.Curve = nil m.RevocationPoint.Curve = nil m.PaymentPoint.Curve = nil m.DelayedPaymentPoint.Curve = nil + m.HtlcPoint.Curve = nil m.FirstCommitmentPoint.Curve = nil case *lnwire.FundingLocked: m.NextPerCommitmentPoint.Curve = nil