Improve channel manager constructor considering RGS

lightningdevkit / ldk-garbagecollected

LDK Bindings for Garbage-Collected Languages

Other

54 stars 27 forks source link

Open TheBlueMatt opened 1 year ago

jkczyz commented 1 year ago

A couple things that I noticed:

Disabling P2PGossipSync requires passing null for net_graph_serialized to the constructor.
But this will start the BackgroundProcessor with GossipSync.none(), which means any NetworkUpdates from failed payments won't be applied to the NetworkGraph

jkczyz commented 1 year ago

Also:

InvoicePayer won't be initialized without a NetworkGraph, making the previous point irrelevant since there would be no event interception