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

⚡️ C-Lightning - GordianServer-macOS-v0.1.4 #54

Closed
wants to merge 16 commits into from

Conversation

Fonta1n3
Copy link
Contributor

@Fonta1n3 Fonta1n3 commented Sep 17, 2020

⚡️ This PR adds the ability to install C-Lightning v0.9.1, along with various other improvements and includes a notarized and pgp signed dmg.

⚠️ This is a minimum viable product, it does not make any effort to verify gpg signatures during install, yet.

All long running tasks are now performed on a background thread which should improve general app performance.

Along with installing c-lightning the user may start, stop and connect remotely to their c-lightning node via a QuickConnect QR code, this functionality can be tested with Fully Noded or via the terminal and in the near future GordianWallet.

To access lightning-cli manually:

$ cd ~/.standup/lightning
$ cli/lightning-cli help

Users may also easily view their lightning log and config via the settings button.

This has been tested on macOS Catalina and Mojave, on a fresh VMWare and my personal desktop.

⚠️ Please note you will not be able to install c-lightning unless bitcoind is installed. You will not be able to start c-lightning unless bitcoind mainnet is running. The hidden service for connecting to c-lightning remotely is configured in the main standup script, therefore in order to test remote connection please delete your ~\.standup directory and go through the install process from scratch.

The install script installs this plugin for exposing lightning-cli to an http server which is exposed to localhost and a Tor V3 hidden service which controls port 1312.

- invalidate the refresh timer whenever we go to the installer to keep tasks minimal
- make the env and lightning config dynamic
- add lightning http hidden service to standup.command
- add configuration script for lightning and the http plugin
 - now will successfully install and configure c-lightning for QuickConnect
- reckless, unfairly cheap, ludicrously fast
@Fonta1n3 Fonta1n3 changed the title ⚡️⚡️⚡️ C-Lightning ⚡️ C-Lightning - GordianServer-macOS-v0.1.4 Sep 17, 2020
- fix QuickConnect bug for c-lightning
- install latest tagged release 0.9.1 for c-lightning
- try and reduce the rainbow spinner when install scripts complete, not sure what is going on here...
- ensure the log is deleted when standing down
- force remove .standup directory when using the kill switch
- fix creation of hidden services for lightning in standup.command script
- fix typo on rpcwhitelist
- add an updated dmg
@Fonta1n3
Copy link
Contributor Author

@ChristopherA this is ready for review.

@ChristopherA
Copy link
Contributor

I did successfully build on on macOS Catalina (10.15.6), using Xcode 11.7.

I did not re-test install full node, but I did try the install c-lightning button. I did not go to the C-Lightning settings (testing that default setup would work).

At end of install (it looked in everything was installed), progress window disappeared with no error.

However, in C-Lightning section the "Install" button is back, no details or active QuickConnect button

At tail of the tail log I have:

Sep 23 00:23:33.000 [notice] New control connection opened from 127.0.0.1.
Sep 23 00:23:40.000 [notice] New control connection opened from 127.0.0.1.
Sep 23 00:23:42.000 [notice] Have tried resolving or connecting to address '[scrubbed]' at 3 different places. Giving up.
Sep 23 00:24:34.000 [notice] Have tried resolving or connecting to address '[scrubbed]' at 3 different places. Giving up.
Sep 23 00:25:17.000 [notice] Have tried resolving or connecting to address '[scrubbed]' at 3 different places. Giving up.
Sep 23 00:27:04.000 [notice] Have tried resolving or connecting to address '[scrubbed]' at 3 different places. Giving up.
Sep 23 00:28:13.000 [notice] Have tried resolving or connecting to address '[scrubbed]' at 3 different places. Giving up.

Two problems: that no error was given after install progress window went away. Then whatever error is causing this problem.

I am emailing full log to @Fonta1n3

@Fonta1n3
Copy link
Contributor Author

Fonta1n3 commented Sep 23, 2020

@ChristopherA bitcoin mainnet must be running or you will not have the option to start lightning.

And that looks like the tor log?

@ChristopherA
Copy link
Contributor

Oops, did send wrong log. LIghtining Log is empty. Is there an install log?

I do have Bitcoin Core Full running, and it is fully synced.

@Fonta1n3
Copy link
Contributor Author

The issue is it is required (as the PR stands) that you need to install everything from scratch as the lightning hidden service’s are configured at the same time as the Bitcoin Core hidden service’s. I will make it more backward compatible so a fresh install is not required 👍

… installing lightning the first time

fix: refresh home page when build scripts complete
@Fonta1n3
Copy link
Contributor Author

Fonta1n3 commented Sep 25, 2020

@ChristopherA the lightning install script has been edited for better backwards compatibility and is now ready for testing and merging. Having said that for testing purposes its always best to start from scratch.

@Fonta1n3
Copy link
Contributor Author

Fonta1n3 commented Oct 6, 2020

@ChristopherA This was the comment I received regarding the specific issue you faced for building c-lightning:

"Hi, I'm writing you because I see issue #4098 and I saw some error like that inside a build procedure in my machine when the libwally was updated, sometimes is necessary remove by hand the build directory inside the external directory. In my case is called "x86_64-linux-gnu"

…andup directory and opening via the console app

fix: add reckless button for c-lightning

fix: cut off text on authentication prompt
@Fonta1n3
Copy link
Contributor Author

Fonta1n3 commented Oct 6, 2020

@ChristopherA I have improved the UX around the log file, added a #reckless button for c-lightning installation and the log now lives in ~/.standup/standup.log as well as fixed the authentication blurb from being cut off.

To test there is no need to "stand down" but I do recommend deleting the ~/.standup directory for testing which will not effect your hidden services or node.

… install, automatically start tor when standup.sh completes to prevent issues where hidden services do not get created if the user does not start Tor, hide c-lightning unless tor is running and Bitcoin Core is running, only attempt to start lightning if the node is fully synced, stope showing Tor V3 auth prompt as it confuses people, hide c-lightning
@Fonta1n3
Copy link
Contributor Author

Fonta1n3 commented Oct 8, 2020

3b9fb61 fixes #57 #58 #59

Known issue: sometimes when the install script finishes it does not want to exit, especially when we automatically launch Tor from the script (which the PR will now do to prevent other more serious issues). If you get the spinning rainbow of death after the install scripts complete you can force quit and reopen the app and all will be fine.

@Fonta1n3 Fonta1n3 closed this Oct 8, 2020
@ChristopherA
Copy link
Contributor

After the C-Lightning installed in #54, I had to do a force-quit, but I did let my VM chug along to complete it's pruned node. It took about two days. However, once it was fully synced, though the C-Lighting pane said "Start", if you clicked it it would percolate for a little bit than nothing. No error message.

I'll save this VM as well as the ~/Library/Application\ Support/Bitcoin files offline. in order to not have to wait two days try to test #60 to later to see if this problem still happens in that PR as well.

@ChristopherA ChristopherA mentioned this pull request Oct 11, 2020
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.

2 participants