diff --git a/common/route.c b/common/route.c index 63a881bf7907..a772aaa1d95c 100644 --- a/common/route.c +++ b/common/route.c @@ -107,7 +107,9 @@ static bool dijkstra_to_hops(struct route_hop **hops, /* Find other end of channel. */ next = gossmap_nth_node(gossmap, c, !(*hops)[num_hops].direction); gossmap_node_get_id(gossmap, next, &(*hops)[num_hops].node_id); - if (gossmap_node_get_feature(gossmap, next, OPT_VAR_ONION) != -1) + /* If we don't have a node_announcement, we *assume* modern */ + if (next->nann_off == 0 + || gossmap_node_get_feature(gossmap, next, OPT_VAR_ONION) != -1) (*hops)[num_hops].style = ROUTE_HOP_TLV; else (*hops)[num_hops].style = ROUTE_HOP_LEGACY; diff --git a/tests/test_pay.py b/tests/test_pay.py index e39af35e3027..6f73e788cdec 100644 --- a/tests/test_pay.py +++ b/tests/test_pay.py @@ -2464,6 +2464,8 @@ def test_error_returns_blockheight(node_factory, bitcoind): @pytest.mark.developer('Needs dev-routes') def test_tlv_or_legacy(node_factory, bitcoind): + # Ideally we'd test with l2 NOT var-onion, but then it can no longer connect + # to us! l1, l2, l3 = node_factory.line_graph(3, opts={'plugin': os.path.join(os.getcwd(), 'tests/plugins/print_htlc_onion.py')}) @@ -2482,8 +2484,8 @@ def test_tlv_or_legacy(node_factory, bitcoind): 'cltv_expiry_delta': 6}]]})['bolt11'] l1.rpc.pay(inv) - # Since L1 hasn't seen broadcast, it doesn't know L2 is TLV, but invoice tells it about L3 - l2.daemon.wait_for_log("Got onion.*'type': 'legacy'") + # Since L1 hasn't seen broadcast, it doesn't know L2 isn't TLV, but invoice tells it about L3 + l2.daemon.wait_for_log("Got onion.*'type': 'tlv'") l3.daemon.wait_for_log("Got onion.*'type': 'tlv'") # We need 5 more blocks to announce l1->l2 channel.