search

Kixunil / tonic_lnd

Rust implementation of LND RPC client using async GRPC library `tonic`
31 stars 44 forks source link

Add support for full LND RPC API #20

Open yzernik opened 2 years ago

yzernik commented 2 years ago

This PR adds support for all of the LND RPC API's (https://github.com/Kixunil/tonic_lnd/issues/15).

I also switched from using rustls to openssl for the TLS certificate handling, because of this issue.

I added another example program in the examples directory showing how to use the add_hold_invoice RPC method in invoicesrpc, as an example of a non-lnrpc RPC call.

This may change the MSRV, not sure if that is important. I have published this fork as a crate if anyone wants to use it already: https://crates.io/crates/tonic_openssl_lnd

grunch commented 2 years ago

I just tested it from https://crates.io/crates/tonic_openssl_lnd and it works great, I think this is big add, thank you @yzernik

yzernik commented 2 years ago

@Kixunil I updated the PR to revert some of the unnecessary changes I made to the connect function signature, so the diff should be much smaller now.

DanGould commented 2 years ago

What a monster PR Jonathan. This is awesome. I will use walletrpc. But it's apparently going to take some time to review

okjodom commented 1 year ago

Appreciating the effort by @yzernik in this PR, and I'm interested in moving the work forward so I can take a dependency (experimental) on tonic_lnd for Fedimint work. I propose we: