lightwalletd icon indicating copy to clipboard operation
lightwalletd copied to clipboard
verified publisher icon zcash

Refactor commandline options for TLS

Open benzcash opened this issue 5 years ago • 1 comments

We made a few changes to TLS config option handling and now the certificate generation isn't reachable anymore.

Should we just remove that code or refactor to the following logic?

  1. No flag, default, no TLS, serve plaintext, print warning
  2. --use-tls flag, if provided (but no cert/key flags) generate certs, print warning
  3. --use-tls and --cert and --key flags, use them, or panic if they don't work

benzcash avatar Mar 24 '20 15:03 benzcash

Sounds good to me, @pacu, @gmale, @mdr0id, do you have an opinion on this?

LarryRuane avatar Mar 24 '20 17:03 LarryRuane