You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently we lean on ChannelManager to build all the reply paths we need to receive replies to our onion messages. As a part of the ongoing OffersFlow work that should change, but one specific feature we should have is the ability to somehow build a reply path from outside of ChannelManager. I currently want this because I was trying to implement DNSSEC proof resolution outside of ChannelManager (to drive conversion of HRNs to offers from outside of LDK "core") and ran into this blocking me.
Hi Matt!
We were brainstorming some ideas around this and wanted to get your thoughts on something.
As we understand it, creating a reply_path mainly involves two pieces:
Selecting a set of peers for the blinded path, and
Using MessageRouter::create_blinded_path to build the path using those peers.
Since MessageRouter is already a trait decoupled from ChannelManager, it seems like — in principle — one could implement this outside ChannelManager, assuming they have their own peer storage and a way to retrieve a suitable Vec<Pubkey>.
So we were wondering — is there something specific that ChannelManager is doing beyond just peer selection and calling create_blinded_path that makes this hard to do externally? Or is the blocker more about how we currently rely on ChannelManager to produce the peer list in practice?
Currently we lean on
ChannelManager
to build all the reply paths we need to receive replies to our onion messages. As a part of the ongoingOffersFlow
work that should change, but one specific feature we should have is the ability to somehow build a reply path from outside ofChannelManager
. I currently want this because I was trying to implement DNSSEC proof resolution outside ofChannelManager
(to drive conversion of HRNs to offers from outside of LDK "core") and ran into this blocking me.cc @shaavan @jkczyz
The text was updated successfully, but these errors were encountered: