From: NMcCloud <kenneth.gendron@gmail.com>
Date: Thu, 9 Mar 2017 04:33:17 +0000 (-0800)
Subject: Remove manager/status?XML=true URI Requirement
X-Git-Tag: ab-debian_0.20170311.01-0ab1~1^2~2^2
X-Git-Url: https://arthur.barton.de/gitweb/?p=netdata.git;a=commitdiff_plain;h=1899f0e3c7d3e67170909be5335d75fb39ac5463

Remove manager/status?XML=true URI Requirement

It would be nice if the end-user could use a different URL to provide the Tomcat status.  Basically you are requiring the manager application be used to provide this, but any app using the org.apache.catalina.manager.StatusManagerServlet servlet is suitable.  I simply do not want to use authentication to provide the information to Netdata.
---

diff --git a/python.d/tomcat.chart.py b/python.d/tomcat.chart.py
index 154e4e4b..c20f85e1 100644
--- a/python.d/tomcat.chart.py
+++ b/python.d/tomcat.chart.py
@@ -51,10 +51,6 @@ class Service(UrlService):
         self.definitions = CHARTS
 
     def check(self):
-        if not self.url.endswith('manager/status?XML=true'):
-            self.error('Bad url(%s). Must be http://<ip.address>:<port>/manager/status?XML=true' % self.url)
-            return False
-
         netloc = urlparse(self.url).netloc.rpartition(':')
         if netloc[1] == ':': port = netloc[2]
         else: port = 80