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

Migrate lnd grpc library #325

Closed
grunch opened this issue Jul 8, 2024 · 4 comments
Closed

Migrate lnd grpc library #325

grunch opened this issue Jul 8, 2024 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@grunch
Copy link
Member

grunch commented Jul 8, 2024

In order to keep Mostro secure, we need to migrate to a well maintained library, after research about the options we have I am leaning towards tonic_lnd which is the original repository of the fork we are using now, so the migration shouldn't be so traumatic.

To be clear we need to replace the dependency tonic_openssl_lnd in favor of tonic_lnd

@grunch grunch added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Jul 8, 2024
@grunch
Copy link
Member Author

grunch commented Jul 9, 2024

For this PR we have a blocker, still we can start working on this one

let's follow this PR Kixunil/tonic_lnd#24

@arkanoider
Copy link
Collaborator

Hold my beer...will do this!

@arkanoider
Copy link
Collaborator

Started testing of library update via #364

@grunch
Copy link
Member Author

grunch commented Oct 17, 2024

closing it via #364

@grunch grunch closed this as completed Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants