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

[WIP] Updates to ilp lib #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Updates to ilp lib #137

wants to merge 1 commit into from

Conversation

adrianhopebailie
Copy link
Contributor

@d1no007 @austin-king @karzak @kincaidoneil @emschwartz

These are the updates I was talking about on the call today.

I have tried to organize things a little better.

  • Defined a type hierarchy and provided both types and then also functions to (de)serialize between them and JSON for SPSP message purposes.
  • This way there is the concept of a Receiver which is a stateful invoice that wraps a STREAM server and SPSP servers become a specialization of that.
  • The express extension is an approach to easily hosting an SPSP server....
  • I started defining some functions that made sense for different use cases in protocol.ts
  • I created a concept of a client that is what ultimately led me to trying to implement net.socket over STREAM and to these experiements: https://github.com/adrianhopebailie/ilp/tree/master/experiments

I recall now why I needed to figure out protocol for the sender and receiver to communicate over STREAM; it was to negotiate things like the payee details etc that we use SPSP for today.

i.e. Assuming you have a secured STREAM connection (you could have exchange addresses and secrets via some other channel) how do you now exchange all the other info like an invoice or requested amount.

STREAM doesn't make it easy to send a fixed receive amount to the recipient so this protocol could be used to set that up (figure out the current rate and an allowable amount of slippage once you start streaming).

Hope some of this is helpful, a lot is a bit icky and unfinished

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

Successfully merging this pull request may close these issues.

None yet

1 participant