Connecting with CLN peer fails with warning

[tanx]

I ran into this issue while attempting to connect with a CLN based LSP peer on testnet (no channel open, just peering). I currently get this warning message when I try to connect to the peer. I spoke with the vendor and they have been able to connect from another LDK based node, so they suggested it might be an issue in the react-native module. This is the config I'm running: tanx@a7b01bf. Thank you 🙏

LDK: Finished noise handshake for connection with 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e
 LOG  react-native-ldk: Success: add_peer_success
 LOG  addPeer Responses: ["add_peer_success"]
 LOG  Node ID: 02ebdcef4e8ce903da7555a00dd607b1d3aa77dbcd4eb3ec0b730c2bc6311cf4ce
 LOG  Backup updated for account wallet0
 LOG  LDK: Received peer Init message from 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e: DataLossProtect: supported, InitialRoutingSync: not supported, UpfrontShutdownScript: supported, GossipQueries: supported, VariableLengthOnion: required, StaticRemoteKey: supported, PaymentSecret: required, BasicMPP: supported, Wumbo: not supported, ShutdownAnySegwit: supported, OnionMessages: not supported, ChannelType: supported, SCIDPrivacy: supported, ZeroConf: supported, unknown flags: supported
 LOG  LDK: Generating channel_reestablish events for 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e
 LOG  react-native-ldk: Persisted channel manager to disk
 LOG  LDK: Got warning message from 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e: gossip_timestamp_filter for bad chain: 0109000000000933ea01ad0ee984209779baaec3ced90fa3f408719526f8d77f494363b44b82ffffffff
 LOG  LDK: Marking channels with 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e disconnected and generating channel_updates. We believe we can make future connections to this peer.

[Jasonvdb]

Are you able to replicate this on regtest using something like Polar?

[tanx]

Well with regtest on polar I can't connect to the CLN peer either, only the LND peer. Calling listpeers also only displays the pubkey of the LND peer.

[tanx]

@Jasonvdb can you confirm that connecting to a CLN peer does not work for you either?

[tnull]

LOG LDK: Got warning message from 025804d4431ad05b06a1a1ee41f22fefeb8ce800b0be3a92ff3b9f594a263da34e: gossip_timestamp_filter for bad chain: 0109000000000933ea01ad0ee984209779baaec3ced90fa3f408719526f8d77f494363b44b82ffffffff

This looks like it is the correct genesis hash for Testnet, however, it's likely in wrong endianess, as I don't see where the byte order is swapped. @Jasonvdb could you check whether you're handing the genesis hash in the right order to LDK?

[Jasonvdb]

I'm testing this on a local regtest and I'm not able to get it to add CLN as a peer. I get a true result from peerHandler.connect(...) in swift but the peer is never added. Nothing in the logs also.

@tnull I don't get the bad chain warning though. I'm not changing the byte order anywhere except for funding_txid where I noticed I had to reverse bytes. Would that be a problem specifically for CLN though? Because we can add LND and Eclair as a peer fine.

[tnull]

@tnull I don't get the bad chain warning though. I'm not changing the byte order anywhere except for funding_txid where I noticed I had to reverse bytes. Would that be a problem specifically for CLN though? Because we can add LND and Eclair as a peer fine.

Well, likely only CLN chooses to close the connection upon receiving a bogus chain hash. LDK does not, not sure about LND. Note this is in line with BOLT 1 as either is allowed:

• upon receiving networks containing no common chains
  - MAY close the connection.

[tanx]

I'm testing this on a local regtest and I'm not able to get it to add CLN as a peer. I get a true result from peerHandler.connect(...) in swift but the peer is never added. Nothing in the logs also.

@tnull I don't get the bad chain warning though. I'm not changing the byte order anywhere except for funding_txid where I noticed I had to reverse bytes. Would that be a problem specifically for CLN though? Because we can add LND and Eclair as a peer fine.

@Jasonvdb thanks for verifying. So it indeed looks like there is a reproducable issue in the react-native module that prevents connecting to a CLN node.

Well, likely only CLN chooses to close the connection upon receiving a bogus chain hash. LDK does not, not sure about LND. Note this is in line with BOLT 1 as either is allowed:

@tnull mmh, if CLN ist following the spec a little more strictly then perhaps the bug just isn't triggered with LND peers?

@Jasonvdb how can I help in debugging the root cause?

[tnull]

@tnull mmh, if CLN ist following the spec a little more strictly then perhaps the bug just isn't triggered with LND peers?

Right, this is likely what happens.

@Jasonvdb how can I help in debugging the root cause?

I think you could try to find where the network graph is setup and byte-swap the genesis hash before it is passed in.

Note we'll also mitigate the issue upstream with the next release by replacing the genesis hash parameter with Network, which doesn't allow for any confusion over byte order (see lightningdevkit/rust-lightning#2025)

[Jasonvdb]

Tested this quickly and reversing bytes of genesis hash doesn't seem to help. Can still connect to other nodes but not CLN.

I also noticed I needed to reverse currentBlockchainTipHash when initialising a new node. I'll add these in #103 but I still wasn't able to connect to a CLN node.

[tnull]

Tested this quickly and reversing bytes of genesis hash doesn't seem to help. Can still connect to other nodes but not CLN.

I also noticed I needed to reverse currentBlockchainTipHash when initialising a new node. I'll add these in #103 but I still wasn't able to connect to a CLN node.

Hm, does it always fail with above mentioned message?

[tanx]

Hey, just wanted to follow up and ask if there are any updates on this? Thanks 🙏

[gkrizek]

I know that Mutiny Wallet is able to peer with CLN just fine and it's based on LDK. Not sure if any of their code could be helpful
https://github.com/MutinyWallet/mutiny-web
https://github.com/MutinyWallet/mutiny-node

[Jasonvdb]

Sorry for the delay, I'll dive into this properly next week

[tnull]

@Jasonvdb Any progress on this? Happy to help debug anything on the LDK side.

[tanx]

Just rebased my config on top of the current master in tanx@10b2c37 and still having this issue.