Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add forwarding channel to HTLC info #99

Open
rkfg opened this issue Oct 13, 2022 · 0 comments
Open

Add forwarding channel to HTLC info #99

rkfg opened this issue Oct 13, 2022 · 0 comments

Comments

@rkfg
Copy link
Contributor

rkfg commented Oct 13, 2022

It's just a field in the protocol with the corresponding channel id (outgoing id for incoming HTLCs and incoming id for outgoing HTLCs), it's currently not copied to the model structure. The idea is to show it in the detailed channel information together with the other HTLC data. I'd need to test a few variants but most importantly I want to see the source or target node name. I think the specific channel id isn't that important because most nodes have just one channel per peer. I have some ideas about the way to show it. Maybe with arrows or another line (like From: <node alias> or To: <node alias> or ⇒ <node alias>, or maybe replace the Incoming: true/false line altogether, never liked it tbh even though I wrote it, etc.)

We have all channels indexed by chanpoint but not id. Maybe adding a second index would be worth it, it should take even less memory because it'd be a uint64->pointer map.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant