Removed extra url variable for the existence check and added CHANGELOG feature line as well as removed #required tag in the sample config

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
 

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"

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
 	}