chore(tracing): deprecates FilterRequestsOnUrl [3.0]

ddtrace/contrib/tornado/__init__.py

@@ -76,11 +76,6 @@ def log_exception(self, typ, value, tb):
             'default_service': 'my-tornado-app',
             'tags': {'env': 'production'},
             'distributed_tracing': False,
-            'settings': {
-                'FILTERS':  [
-                    FilterRequestsOnUrl(r'http://test\\.example\\.com'),
-                ],
-            },
         },
     }
 

ddtrace/filters.py

@@ -4,7 +4,7 @@
 
 
 deprecate(
-    "The ddtrace.filters module is deprecated and will be removed.",
-    message="Import ``TraceFilter`` and/or ``FilterRequestsOnUrl`` from the ddtrace.trace package.",
+    "The ddtrace.filters module and the ``FilterRequestsOnUrl`` class is deprecated and will be removed.",
+    message="Import ``TraceFilter`` from the ddtrace.trace package.",
     category=DDTraceDeprecationWarning,
 )

ddtrace/trace/__init__.py

@@ -1,8 +1,7 @@
 from ddtrace._trace.context import Context
-from ddtrace._trace.filters import FilterRequestsOnUrl
 from ddtrace._trace.filters import TraceFilter
 from ddtrace._trace.pin import Pin
 
 
 # TODO: Move `ddtrace.Tracer`, `ddtrace.Span`, and `ddtrace.tracer` to this module
-__all__ = ["Context", "Pin", "TraceFilter", "FilterRequestsOnUrl"]
+__all__ = ["Context", "Pin", "TraceFilter"]

docs/advanced_usage.rst

@@ -332,24 +332,25 @@ configuring the tracer with a filters list. For instance, to filter out
 all traces of incoming requests to a specific url::
 
     from ddtrace import tracer
+    from ddtrace.trace import TraceFilter
+
+    class FilterbyName(TraceFilter):
+        def process_trace(self, trace):
+            for span in trace:
+                if span.name == "some_name"
+                    # drop the full trace chunk
+                    return None
+            return trace
 
     tracer.configure(settings={
         'FILTERS': [
-            FilterRequestsOnUrl(r'http://test\.example\.com'),
+            FilterbyName(),
         ],
     })
 
 The filters in the filters list will be applied sequentially to each trace
 and the resulting trace will either be sent to the Agent or discarded.
 
-**Built-in filters**
-
-The library comes with a ``FilterRequestsOnUrl`` filter that can be used to
-filter out incoming requests to specific urls:
-
-.. autoclass:: ddtrace.trace.FilterRequestsOnUrl
-    :members:
-
 **Writing a custom filter**
 
 Create a filter by implementing a class with a ``process_trace`` method and

releasenotes/notes/ddtrace-resourcefilter-deprecated-52b1c92d388b0518.yaml

@@ -0,0 +1,4 @@
+---
+deprecations:
+  - |
+    tracing: Deprecates ``ddtrace.filters.FilterRequestsOnUrl``. Spans should be filtered/sampled using DD_TRACE_SAMPLING_RULES configuration.

tests/tracer/test_filters.py

@@ -2,9 +2,9 @@
 
 import pytest
 
+from ddtrace._trace.filters import FilterRequestsOnUrl
 from ddtrace._trace.span import Span
 from ddtrace.ext.http import URL
-from ddtrace.trace import FilterRequestsOnUrl
 from ddtrace.trace import TraceFilter