skyline
skyline copied to clipboard
Allow for HTTPS Graphite servers.
If a settings.GRAPHITE_HOST contains an HTTPS Graphite server, this replacement fails. Using a regular expression to allow for both http and https URI protocols.
In the future, perhaps this interpretation should live in settings, rather than in each place in the code?
The webapp javascript needs the scheme wholesale. We could possibly add another setting called GRAPHITE_HOST_SCHEME and use that both here and in the webapp javascript, but I think it's a pretty minor distinction and I'd rather not break folk's settings configurations for it.
:+1: