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

plugins: rust clnrest plugin #7541

Closed
wants to merge 1 commit into from

Conversation

JosephGoulden
Copy link
Contributor

I'm running clightning on Nixos and make use of the REST API. The python build of the clnrest plugin keeps breaking because of incorrect dependency versions in nixpkgs (from what I understand python on nix is a bit of a pain, and its not a language I use so I don't care to dig into it).

So I wrote a replacement plugin in rust that utilises the cln-rpc and cln-plugin crates and it is working well.

I'm not aware of the architecture/language strategy for clightning and I don't mean to tread on any toes so I'm just opening this draft PR to see if this is something that would be of interest, or perhaps it should be in a different repo?

@daywalker90
Copy link
Contributor

Duplicate of #7509

@JosephGoulden
Copy link
Contributor Author

Nice one @daywalker90 !

@chrisguida
Copy link
Contributor

Wow, great minds think alike!

@chrisguida
Copy link
Contributor

@JosephGoulden does this plugin have any features missing from @daywalker90's plugin?

Anyway it would be amazing if you could help test/maintain the other plugin! :)

@JosephGoulden
Copy link
Contributor Author

@chrisguida No the other PR was already further down the line with features and integration into this project.

It would be a lot easier for me to work on this project if we could get #6650 merged. Even just a very basic flake that we can iterate on would be good. Happy to help with any rust plugins I can make use of (currently trying a few out).

@chrisguida
Copy link
Contributor

Awesome! Yeah #6650 is just a draft, not sure if I ever got it working... did you try it yet?

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.

3 participants