Browse Source

Revert "[917] split XXX_URL_PATH into BASE_URL_PATH and XXX_URL_PATH"

This reverts commit a458d69d94b2a7506395e7a24e5c2a9146791553.
Naoki Kambe 13 years ago
parent
commit
61dd61b8f2
2 changed files with 25 additions and 26 deletions
  1. 11 12
      src/bin/stats/stats_httpd.py.in
  2. 14 14
      src/bin/stats/tests/b10-stats-httpd_test.py

+ 11 - 12
src/bin/stats/stats_httpd.py.in

@@ -61,13 +61,12 @@ XSD_TEMPLATE_LOCATION = BASE_LOCATION + os.sep + "stats-httpd-xsd.tpl"
 XSL_TEMPLATE_LOCATION = BASE_LOCATION + os.sep + "stats-httpd-xsl.tpl"
 
 # These variables are paths part of URL.
-# eg. "http://${address}" + BASE_URL_PATH + ( | /{module_name} | /{module_name}/{item_name} ) + XXX_URL_PATH
-BASE_URL_PATH = '/bind10/statistics'
-XML_URL_PATH = '/xml'
-XSD_URL_PATH = '/xsd'
-XSL_URL_PATH = '/xsl'
+# eg. "http://${address}" + XXX_URL_PATH
+XML_URL_PATH = '/bind10/statistics/xml'
+XSD_URL_PATH = '/bind10/statistics/xsd'
+XSL_URL_PATH = '/bind10/statistics/xsl'
 # TODO: This should be considered later.
-XSD_NAMESPACE = 'http://bind10.isc.org' + BASE_URL_PATH + XSD_URL_PATH
+XSD_NAMESPACE = 'http://bind10.isc.org' + XSD_URL_PATH
 
 # Assign this process name
 isc.util.process.rename()
@@ -86,11 +85,11 @@ class HttpHandler(http.server.BaseHTTPRequestHandler):
 
     def send_head(self):
         try:
-            if self.path == BASE_URL_PATH + XML_URL_PATH:
+            if self.path == XML_URL_PATH:
                 body = self.server.xml_handler()
-            elif self.path == BASE_URL_PATH + XSD_URL_PATH:
+            elif self.path == XSD_URL_PATH:
                 body = self.server.xsd_handler()
-            elif self.path == BASE_URL_PATH + XSL_URL_PATH:
+            elif self.path == XSL_URL_PATH:
                 body = self.server.xsl_handler()
             else:
                 if self.path == '/' and 'Host' in self.headers.keys():
@@ -98,7 +97,7 @@ class HttpHandler(http.server.BaseHTTPRequestHandler):
                     self.send_response(302)
                     self.send_header(
                         "Location",
-                        "http://" + self.headers.get('Host') + BASE_URL_PATH + XML_URL_PATH)
+                        "http://" + self.headers.get('Host') + XML_URL_PATH)
                     self.end_headers()
                     return None
                 else:
@@ -396,8 +395,8 @@ class StatsHttpd:
         self.xml_body = self.open_template(XML_TEMPLATE_LOCATION).substitute(
             xml_string=xml_string,
             xsd_namespace=XSD_NAMESPACE,
-            xsd_url_path=BASE_URL_PATH + XSD_URL_PATH,
-            xsl_url_path=BASE_URL_PATH + XSL_URL_PATH)
+            xsd_url_path=XSD_URL_PATH,
+            xsl_url_path=XSL_URL_PATH)
         assert self.xml_body is not None
         return self.xml_body
 

+ 14 - 14
src/bin/stats/tests/b10-stats-httpd_test.py

@@ -135,7 +135,7 @@ class TestHttpHandler(unittest.TestCase):
         self.assertEqual((self.address, self.port), self.stats_httpd.http_addrs[0])
 
         # URL is '/bind10/statistics/xml'
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XML_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XML_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.getheader("Content-type"), "text/xml")
@@ -145,13 +145,13 @@ class TestHttpHandler(unittest.TestCase):
         self.assertTrue(root.tag.find('stats_data') > 0)
         for (k,v) in root.attrib.items():
             if k.find('schemaLocation') > 0:
-                self.assertEqual(v, stats_httpd.XSD_NAMESPACE + ' ' + stats_httpd.BASE_URL_PATH + stats_httpd.XSD_URL_PATH)
+                self.assertEqual(v, stats_httpd.XSD_NAMESPACE + ' ' + stats_httpd.XSD_URL_PATH)
         for mod in DUMMY_DATA:
             for (item, value) in DUMMY_DATA[mod].items():
                 self.assertIsNotNone(root.find(mod + '/' + item))
 
         # URL is '/bind10/statitics/xsd'
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSD_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSD_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.getheader("Content-type"), "text/xml")
@@ -171,7 +171,7 @@ class TestHttpHandler(unittest.TestCase):
             self.assertTrue(elm.attrib['name'] in DUMMY_DATA)
 
         # URL is '/bind10/statitics/xsl'
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSL_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSL_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.getheader("Content-type"), "text/xml")
@@ -205,7 +205,7 @@ class TestHttpHandler(unittest.TestCase):
         response = self.client.getresponse()
         self.assertEqual(response.status, 302)
         self.assertEqual(response.getheader('Location'),
-                         "http://%s:%d%s" % (self.address, self.port, (stats_httpd.BASE_URL_PATH + stats_httpd.XML_URL_PATH)))
+                         "http://%s:%d%s" % (self.address, self.port, stats_httpd.XML_URL_PATH))
 
         # 404 NotFound
         self.client._http_vsn_str = 'HTTP/1.0'
@@ -226,19 +226,19 @@ class TestHttpHandler(unittest.TestCase):
         self.stats_httpd.cc_session.set_timeout(milliseconds=100)
 
         # request XML
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XML_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XML_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
 
         # request XSD
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSD_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSD_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
 
         # request XSL
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSL_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSL_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
@@ -251,25 +251,25 @@ class TestHttpHandler(unittest.TestCase):
             )
 
         # request XML
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XML_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XML_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
 
         # request XSD
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSD_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSD_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
 
         # request XSL
-        self.client.putrequest('GET', stats_httpd.BASE_URL_PATH + stats_httpd.XSL_URL_PATH)
+        self.client.putrequest('GET', stats_httpd.XSL_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 500)
 
     def test_do_HEAD(self):
-        self.client.putrequest('HEAD', stats_httpd.BASE_URL_PATH + stats_httpd.XML_URL_PATH)
+        self.client.putrequest('HEAD', stats_httpd.XML_URL_PATH)
         self.client.endheaders()
         response = self.client.getresponse()
         self.assertEqual(response.status, 200)
@@ -591,8 +591,8 @@ class TestStatsHttpd(unittest.TestCase):
             stats_httpd.XML_TEMPLATE_LOCATION).substitute(
             xml_string='<Dummy><foo>bar</foo></Dummy>',
             xsd_namespace=stats_httpd.XSD_NAMESPACE,
-            xsd_url_path=stats_httpd.BASE_URL_PATH + stats_httpd.XSD_URL_PATH,
-            xsl_url_path=stats_httpd.BASE_URL_PATH + stats_httpd.XSL_URL_PATH)
+            xsd_url_path=stats_httpd.XSD_URL_PATH,
+            xsl_url_path=stats_httpd.XSL_URL_PATH)
         xml_body2 = self.stats_httpd.xml_handler()
         self.assertEqual(type(xml_body1), str)
         self.assertEqual(type(xml_body2), str)