Merge pull request #1093 from romain-dartigues/rdartigues

[netdata.git] / charts.d / tomcat.chart.sh

diff --git a/charts.d/tomcat.chart.sh b/charts.d/tomcat.chart.sh
index b0eddc7f0dac66345b79ad55b017a321b53ae5ac..cc6baea1ae1f967fe2210086d96ffb13ed8611b9 100755 (executable)
--- a/charts.d/tomcat.chart.sh
+++ b/charts.d/tomcat.chart.sh
@@ -1,4 +1,4 @@
-#!/bin/bash
+# no need for shebang - this file is loaded from charts.d.plugin
 
 # Description: Tomcat netdata charts.d plugin
 # Author: Jorge Romero
@@ -9,8 +9,8 @@ tomcat_url=""
 tomcat_curl_opts=""
 
 # set tomcat username/password here
-tomcatUser=""
-tomcatPassword=""
+tomcat_user=""
+tomcat_password=""
 
 # _update_every is a special variable - it holds the number of seconds
 # between the calls of the _update() function
@@ -37,13 +37,23 @@ tomcat_check() {
                echo >&2 "tomcat url is unset or set to the empty string"
                return 1
        fi
-       if [ -z "${tomcatUser}" ]; then
-               echo >&2 "tomcat user is unset or set to the empty string"
-               return 1
+       if [ -z "${tomcat_user}" ]; then
+               # check backwards compatibility
+               if [ -z "${tomcatUser}" ]; then         
+                       echo >&2 "tomcat user is unset or set to the empty string"
+                       return 1
+               else
+                       tomcat_user="${tomcatUser}"
+               fi
        fi
-       if [ -z "${tomcatPassword}" ]; then
-               echo >&2 "tomcat password is unset or set to the empty string"
-               return 1
+       if [ -z "${tomcat_password}" ]; then
+               # check backwards compatibility
+               if [ -z "${tomcatPassword}" ]; then
+                       echo >&2 "tomcat password is unset or set to the empty string"
+                       return 1
+               else
+                       tomcat_password="${tomcatPassword}"
+               fi
        fi
 
        # check if we can get to tomcat's status page
@@ -65,7 +75,7 @@ tomcat_check() {
 tomcat_get() {
        # collect tomcat values
        tomcat_port="$(IFS=/ read -ra a <<< "$tomcat_url"; hostport=${a[2]}; echo "${hostport#*:}")"
-       mapfile -t lines < <(curl -u "$tomcatUser":"$tomcatPassword" -Ss ${tomcat_curl_opts} "$tomcat_url" |\
+       mapfile -t lines < <(curl -u "$tomcat_user":"$tomcat_password" -Ss ${tomcat_curl_opts} "$tomcat_url" |\
                xmlstarlet sel \
                        -t -m "/status/jvm/memory" -v @free \
                        -n -m "/status/connector[@name='\"http-bio-$tomcat_port\"']/threadInfo" -v @currentThreadCount \