chainreg: clarify zmq port-mismatch warnings
What the developer wrote
Authored by Boris Nagaev
chainreg: clarify zmq port-mismatch warnings
The previous warning text ("unable to subscribe to zmq ... events") suggested
that lnd failed to create the ZMQ connection, when in reality it only means the
configured port differs from what bitcoind reports via getzmqnotifications.
Reword both messages to say "port mismatch" and tell to verify the port.
Fixes https://github.com/lightningnetwork/lnd/issues/10568
What changed, and why it matters
This commit only rewords two warning log messages in LND's chain registry. It changes the text from saying 'unable to subscribe to zmq ... events' to saying 'zmq ... port mismatch' when LND's configured ZMQ port differs from what the connected bitcoind reports. There is no code behavior change, no security fix, and no vulnerability addressed.
No security action required. This is a logging/UX clarification only. Operators may benefit from the clearer warning if they misconfigure ZMQ ports, but the change does not alter security posture.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies two log.Warnf calls in chainreg/chainregistry.go. The surrounding logic—checking whether bitcoind’s reported ZMQ notification address port matches LND’s configured zmqpubrawblock/zmqpubrawtx ports and setting pubRawBlockActive/pubRawTxActive—is unchanged. Only the human-readable warning strings are updated to be more accurate and actionable. No functional, cryptographic, or network change is introduced.
Changed components
chainreg/chainregistry.goZMQ port-mismatch warning loggingInspect captured patch +14 / −10
diff --git a/chainreg/chainregistry.go b/chainreg/chainregistry.go
index 55f9244..30a9082 100644
--- a/chainreg/chainregistry.go
+++ b/chainreg/chainregistry.go
@@ -465,12 +465,14 @@ func NewPartialChainControl(cfg *Config) (*PartialChainControl, func(), error) {
if zmq[i].Address.Port() !=
zmqPubRawBlockURL.Port() {
- log.Warnf(
- "unable to subscribe to zmq block events on "+
- "%s (bitcoind is running on %s)",
+ log.Warnf("zmq block port "+
+ "mismatch: lnd is "+
+ "using %s but "+
+ "bitcoind reports %s "+
+ "- ensure the port "+
+ "is correct",
zmqPubRawBlockURL.Host,
- zmq[i].Address.Host,
- )
+ zmq[i].Address.Host)
}
pubRawBlockActive = true
}
@@ -478,12 +480,14 @@ func NewPartialChainControl(cfg *Config) (*PartialChainControl, func(), error) {
if zmq[i].Address.Port() !=
zmqPubRawTxURL.Port() {
- log.Warnf(
- "unable to subscribe to zmq tx events on "+
- "%s (bitcoind is running on %s)",
+ log.Warnf("zmq tx port "+
+ "mismatch: lnd is "+
+ "using %s but "+
+ "bitcoind reports %s "+
+ "- ensure the port "+
+ "is correct",
zmqPubRawTxURL.Host,
- zmq[i].Address.Host,
- )
+ zmq[i].Address.Host)
}
pubRawTxActive = true
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.