lnd icon indicating copy to clipboard operation
lnd copied to clipboard

peer: fix logging (missing argument) [skip ci]

Open starius opened this issue 1 year ago • 0 comments

Change Description

Previously, LND log used to have such records:

[WRN] PEER: Peer(...): Link=%!v(MISSING) is not active

Now the short channel ID is printed there instead of "%!v(MISSING)".

Steps to Test

I got this message when opened a channel from a node using neutrino. I had to restart the node multiple times until the channel become active and I got this message after the first restart.

Also I found this message in the log of another node running with full bitcoin node.

Pull Request Checklist

Testing

  • [ ] Your PR passes all CI checks.
  • [ ] ~~Tests covering the positive and negative (error paths) are included.~~(irrelevant)
  • [ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

📝 Please see our Contribution Guidelines for further guidance.

starius avatar Oct 18 '24 15:10 starius