go-nitro icon indicating copy to clipboard operation
go-nitro copied to clipboard

add HTLCs to ledger channel update protocol

Open NiloCK opened this issue 2 years ago • 4 comments

This is draft work toward including HTLC transfers as part of the base consensus_channel.

Strongest caveat: there is no on-chain implementation to enforce any of these data types.

NiloCK avatar Oct 09 '23 04:10 NiloCK

Deploy Preview for nitrodocs processing.

Name Link
Latest commit a2f9a68fb4634a841981560508ff91e417e07547
Latest deploy log https://app.netlify.com/sites/nitrodocs/deploys/6526c2e181423b0008cf0ad0

netlify[bot] avatar Oct 09 '23 04:10 netlify[bot]

Deploy Preview for nitro-payment-demo canceled.

Name Link
Latest commit a2f9a68fb4634a841981560508ff91e417e07547
Latest deploy log https://app.netlify.com/sites/nitro-payment-demo/deploys/6526c2e12462960008c289b7

netlify[bot] avatar Oct 10 '23 13:10 netlify[bot]

This is looking good so far @NiloCK !

I think we probably want:

  • [ ] an ADR to accompany these changes, explaining how LN-compatible multi hop payments can be incorporated into go-nitro. You've probably got the germ of that in Notion already.
  • [ ] some unit test for the functionality you built so far
  • [ ] eventually an integration test involving any changes to the go-nitro API (etc). We could develop that as a feature branch and merge it all in one go

geoknee avatar Oct 12 '23 08:10 geoknee

:+1:

Will probably shelve this until after the hackathon.

NiloCK avatar Oct 13 '23 13:10 NiloCK