Revert to using https when generating a baseUrl

[ianconsolata]

https is a more secure default than http. Additionally, the previous helper file created https baseUrls, so upgrading to the new chart version will cause the base URL of the server to change if folks don't have baseUrlOverride set.

[ianconsolata]

Not sure what version to use here. This is technically a breaking change, but only because it's fixing an issue with a previous change that was probably unintentional.

Here's the commit that changed it:
5ff47fe

It seems like perhaps it should only be https when folks are using an ingress, though I can't imagine why folks would use this without an ingress so it's hard to reason about what they would want there.

[MisterTimn]

The http/https setup is on my own todo list, atm clusters I use actually all have TLS termination at the proxy level so its a different setup (just needs https in base URL but no need for cert/key/tls secret).

Didn't want https to be the default since it's not default for CSS itself atm, and setup of https varies depending on config, but we should definitely push to make this the default. Having plain http and no ingress can be used when all communication is kept inside the cluster, to develop some internal application.

tls:
  enabled: false
  autoGenerated: true
  existingSecret: ""
  caCert: "" #base64 encoded
  caKey: "" #base64 encoded

Perhaps we could enable it by default and use self signed certificates that get auto generated when key and cert aren't specified? Added option in my example.