Jake Sylvestre
Jake Sylvestre
One more thing I'll add, it does look like lnd has tried this approach (see: lightningnetwork/lnd#2602) in [a few different places](https://github.com/lightningnetwork/lnd/search?q=module) which definitely makes me more comfortable. There's only [one...
@jcvernaleo (as per #1530) - Low priority - Bug(ish)
@jcvernaleo (as per #1530) - Low priority - Enhancement @ledzep2 had the same issue, just commented out the verification on my fork
@Roasbeef would like to get your input here. I know there are some security concerns around allowing users to do this and this is behavior we'd explicitly like to discourage
@jcvernaleo I'm pretty agnostic here. The counterargument here would be something like this: btcd doesn't have a wallet API and is used pretty heavily inside private networks where SSL adds...
As an aside, if we do merge this we'll need to do a force push to get the tests to pass
I'm with @ledzep2 on this one. Since we don't have consensus/@jcvernaleo is opposed as it stands, this isn't getting merged. Would like to get more feedback from @Roasbeef /anyone else...
@jcvernaleo, just an idea here but I'm wondering about a middle zone after reading #1474. We _could_ actually add a helper command to generate the tls cert using golang.
I'll take a look at this. @ledzep2 does this seem like a good solution?
@jcvernaleo (as per #1530) - Low priority - Enhancement - Outdated (needs to implement changes from @torkelrogstad's review)