diff --git a/CHANGELOG.md b/CHANGELOG.md index 9d4d74ff2..b210ea496 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -24,6 +24,7 @@ time before a new metric is included by the plugin. - [#1264](https://github.com/influxdata/telegraf/pull/1264): Add SSL config options to http_response plugin. - [#1272](https://github.com/influxdata/telegraf/pull/1272): graphite parser: add ability to specify multiple tag keys, for consistency with influxdb parser. - [#1265](https://github.com/influxdata/telegraf/pull/1265): Make dns lookups for chrony configurable. Thanks @zbindenren! +- [#1278](https://github.com/influxdata/telegraf/pull/1278): RabbitMQ input: made url parameter optional by using DefaultURL (http://localhost:15672) if not specified ### Bugfixes diff --git a/etc/telegraf.conf b/etc/telegraf.conf index c5d631f51..05f47e861 100644 --- a/etc/telegraf.conf +++ b/etc/telegraf.conf @@ -1176,7 +1176,7 @@ # # Read metrics from one or many RabbitMQ servers via the management API # [[inputs.rabbitmq]] -# url = "http://localhost:15672" # required +# # url = "http://localhost:15672" # # name = "rmq-server-1" # optional tag # # username = "guest" # # password = "guest" diff --git a/plugins/inputs/rabbitmq/rabbitmq.go b/plugins/inputs/rabbitmq/rabbitmq.go index 6d5379438..fbfd2792e 100644 --- a/plugins/inputs/rabbitmq/rabbitmq.go +++ b/plugins/inputs/rabbitmq/rabbitmq.go @@ -146,13 +146,12 @@ func (r *RabbitMQ) Gather(acc telegraf.Accumulator) error { } func (r *RabbitMQ) requestJSON(u string, target interface{}) error { - url := r.URL - if url == "" { - url = DefaultURL + if r.URL == "" { + r.URL = DefaultURL } u = fmt.Sprintf("%s%s", url, u) - req, err := http.NewRequest("GET", u, nil) + req, err := http.NewRequest("GET", r.URL, nil) if err != nil { return err }