htlcswitch: improve loggings for Keystone

htlcswitch/circuit_map.go

@@ -935,7 +935,7 @@ type Keystone struct {
 }
 
 // String returns a human readable description of the Keystone.
-func (k *Keystone) String() string {
+func (k Keystone) String() string {
 	return fmt.Sprintf("%s --> %s", k.InKey, k.OutKey)
 }
 

htlcswitch/mailbox.go

@@ -548,6 +548,8 @@ func (m *memoryMailBox) mailCourier(cType courierType) {
 				m.pktCond.L.Unlock()
 
 			case <-deadline:
+				log.Debugf("Expiring add htlc with "+
+					"keystone=%v", add.keystone())
 				m.FailAdd(add)
 
 			case pktDone := <-m.pktReset:

htlcswitch/mailbox_test.go

@@ -294,7 +294,7 @@ func (c *mailboxContext) checkFails(adds []*htlcPacket) {
 
 	select {
 	case pkt := <-c.forwards:
-		c.t.Fatalf("unexpected forward: %v", pkt)
+		c.t.Fatalf("unexpected forward: %v", pkt.keystone())
 	case <-time.After(50 * time.Millisecond):
 	}
 }