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

Faraday #219

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Faraday #219

wants to merge 2 commits into from

Conversation

sputn1ck
Copy link
Contributor

@sputn1ck sputn1ck commented Aug 5, 2020

This PR adds Faraday. Faraday is a Lightning Channel Management & Optimization Tool by Lightning Labs. It is used in Lightning Terminal which could be pretty useful in the future.

I did this PR based on #209 as I have never created modules and pkgs myself. I have not tested the netns stuff and am not sure about the tests.

@nixbitcoin
Copy link
Member

I fixed up your PR, separated the add pkg and module commit, added functional tests, and added a lightning-loop enhancement on top. We need this enhancement so LiT can eventually communicate with lightning-loop with netns-isolation enabled. https://github.com/nixbitcoin/nix-bitcoin/commits/faraday

We also need a Tor proxy capability in Faraday as described in lightninglabs/faraday#37 (comment) so we can keep netns-level Tor enforcement.

@sputn1ck sputn1ck force-pushed the faraday branch 2 times, most recently from 0c82e25 to 6688a71 Compare February 1, 2021 20:46
@erikarvstedt
Copy link
Collaborator

Please squash the commit history so that it's ready for review and add a test.

@sputn1ck
Copy link
Contributor Author

sputn1ck commented Feb 3, 2021

Will update this with tests similiar to #313

@prusnak
Copy link
Contributor

prusnak commented Jun 20, 2021

Faraday is now available in nixpkgs: NixOS/nixpkgs#127333

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.

4 participants