bosun-monitor / bosun

Time Series Alerting Framework
http://bosun.org
MIT License
3.4k stars 495 forks source link

Add support for being able to specify the scheme ( e.g. https ) #2486

Closed johnewing1 closed 3 years ago

johnewing1 commented 4 years ago

Add support for being able to specify the scheme ( e.g. https ) in opentsdb urls


Description

Fixes #2087

Type of change

How has this been tested?

Checklist:

neilfordyce commented 4 years ago

Looks great. Probably worthy of updates to docs and reference configs in a couple of places:

  1. /docs/configuration.md
  2. /cmd/bosun/bosun.example.toml
  3. /docker/data/bosun.toml
johnewing1 commented 3 years ago

Looks great. Probably worthy of updates to docs and reference configs in a couple of places:

  1. /docs/configuration.md
  2. /cmd/bosun/bosun.example.toml
  3. /docker/data/bosun.toml

File 1 is for the legacy 0.5 config, so we should leave that as is. I've updated the other two.