-
Notifications
You must be signed in to change notification settings - Fork 39
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 support for RPC #92
Comments
I have yet to take the time to fully grok the reference RPC implementation; hence the cryptic comments. This is basically first thing in my personal TODO. I'll help fill this out afterward. Also, glad to have the help! I have time right now for code review, but not a whole lot else. |
The current state of this as I understand it:
Relevant mailing list thread: https://groups.google.com/g/capnproto/c/oMQEuOI0xW8 @jdiaz5513, any updates on your end? |
The "somewhere" is presently only on my local machine--I didn't built a self-standing POC for it, but rather directly starting integrating it into my application. I have a capnp-go peer which hosts a web socket server which is connected to by capnp-ts. The relevant glue code on the TypeScript side is in this gist, which I believe should actually be part of any capnp-ts library which provides RPC support. The go side of it is really straightforward, but in the interests of completeness here is a sample implementation. Using these two things, and my fork of the repo, I'm able to pass capabilities between the two peers and everything appears to work correctly. |
I haven't seen anything about the state of work on RPC support (other than a few cryptic comments around the repo). It's something I'm interested in, and I am aware it's a pretty sizeable project, so I figured a good concrete step is to try to identify what needs to be done to get there.
Perhaps consider this a tracking issue for the general goal of RPC support.
So, here is my attempt at documenting the steps needed to get there. There is probably a lot of detail that I'm missing, but hopefully we can flesh it out here.
The text was updated successfully, but these errors were encountered: