default to /

Author: florentinl

ddtrace/_trace/processor/resource_renaming.py

@@ -40,7 +40,7 @@ def _compute_simplified_endpoint(self, url: Optional[str]) -> str:
 
         match = self._URL_PATH_EXTRACTION_RE.match(url)
         if not match:
-            return ""
+            return "/"
         path = match.group("path")
         if not path or path == "/":
             return "/"

tests/tracer/test_resource_renaming.py

@@ -52,7 +52,7 @@ def test_compute_simplified_endpoint_path_element(self, elem, expected):
         [
             # Basic cases
             ("", "/"),
-            ("http://example.com", ""),
+            ("http://example.com", "/"),
             ("http://example.com/", "/"),
             ("/users", "/users"),
             ("https://example.com/users", "/users"),
@@ -85,8 +85,7 @@ def test_compute_simplified_endpoint_path_element(self, elem, expected):
             ),
             # Error cases
             (None, "/"),
-            ("invalid-url", ""),
-            ("://malformed", ""),
+            ("://malformed", "/"),
         ],
     )
     def test_compute_simplified_endpoint(self, url, expected):
@@ -143,7 +142,7 @@ def test_processor_malformed_url(self):
         span.set_tag(http.URL, "not-a-valid-url")
 
         processor.on_span_finish(span)
-        assert span.get_tag(http.ENDPOINT) == ""
+        assert span.get_tag(http.ENDPOINT) == "/"
 
     def test_regex_patterns(self):
         processor = ResourceRenamingProcessor()