orbitalturtle
orbitalturtle
026c9656a124ae35962e8b95fd926de703ef9d71d9e61701b294e39b7a95fedd19@t43ezoqjbkw4fqwzeqyyts64dgzjzj6hefrjnyfu4hf6asmttdn5hmid.onion:9911
Thanks @bitromortac! Yes this PR depends on this one: https://github.com/lightningnetwork/lnd/pull/6573, so might be worth giving that one a review first. (Also noticed both PRs need a rebase. Will do that...
@guggero Oops, sorry, looks like I broke something when trying to fix the onion file stuff, as discussed here: https://github.com/lightningnetwork/lnd/pull/4458#discussion_r804571800 But I'm having trouble fixing the issue... First question, if...
@guggero Ah got it, thank you for the explanation!
Hey @guggero one more question for you! Now I've split the Tor code out into a separate PR. But the Tor changes depend on changes in the keychain and lnencrypt...
Got it, thank you @guggero . Will let you know when I have the latest PRs up.
@guggero One more question... so because we're getting rid of all the lnd dependencies in the Tor module, we don't need to split the onionfile code into a new subpackage...
Rebased this just fyi. It's ready for a look!
@Crypt-iQ and @guggero, thanks again for the reviews! I made those updates.
Thanks @guggero! I made those changes. And sure, I'll definitely keep a closer eye out for those nits & code style guidelines in the future!