check excluded tracing paths based on regex

Author: Jonas Miederer

opencensus/trace/utils.py

@@ -64,7 +64,7 @@ def disable_tracing_url(url, excludelist_paths=None):
     url_path = url.split('/', 1)[1]
 
     for path in excludelist_paths:
-        if url_path.startswith(path):
+        if re.match(path, url_path):
             return True
 
     return False

tests/unit/trace/test_ext_utils.py

@@ -58,6 +58,48 @@ def test_disable_tracing_url_explicit(self):
         disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
         self.assertTrue(disable_tracing)
 
+    def test_disable_tracing_url_partial(self):
+        url = 'http://127.0.0.1:8080/test_no_tracing'
+        excludelist_paths = ['test']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertTrue(disable_tracing)
+
+    def test_disable_tracing_url_partial_regex_match(self):
+        url = 'http://127.0.0.1:8080/test_no_tracing'
+        excludelist_paths = ['^test']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertTrue(disable_tracing)
+
+    def test_disable_tracing_url_partial_regex_nomatch(self):
+        url = 'http://127.0.0.1:8080/test_no_tracing'
+        excludelist_paths = ['test$']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertFalse(disable_tracing)
+
+    def test_disable_tracing_url_root(self):
+        url = 'http://127.0.0.1:8080/'
+        excludelist_paths = ['']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertTrue(disable_tracing)
+
+    def test_disable_tracing_url_root_regex(self):
+        url = 'http://127.0.0.1:8080/'
+        excludelist_paths = ['^$']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertTrue(disable_tracing)
+
+    def test_disable_tracing_url_root_empty_exclude(self):
+        url = 'http://127.0.0.1:8080/test_no_tracing'
+        excludelist_paths = ['']
+
+        disable_tracing = utils.disable_tracing_url(url, excludelist_paths)
+        self.assertTrue(disable_tracing)
+
     def test_disable_tracing_hostname_default(self):
         url = '127.0.0.1:8080'