Revert "Merge branch '2.x-staging' into conti/support-aws-api-gateway"

This reverts commit fe7e93f, reversing
changes made to 4ae108b.

Author: wantsui

ddtrace/_trace/pin.py

@@ -6,6 +6,7 @@
 import wrapt
 
 import ddtrace
+from ddtrace.vendor.debtcollector import deprecate
 
 from ..internal.logger import get_logger
 
@@ -31,17 +32,25 @@ class Pin(object):
         >>> conn = sqlite.connect('/tmp/image.db')
     """
 
-    __slots__ = ["tags", "_tracer", "_target", "_config", "_initialized"]
+    __slots__ = ["tags", "tracer", "_target", "_config", "_initialized"]
 
     def __init__(
         self,
         service=None,  # type: Optional[str]
         tags=None,  # type: Optional[Dict[str, str]]
+        tracer=None,
         _config=None,  # type: Optional[Dict[str, Any]]
     ):
         # type: (...) -> None
+        if tracer is not None and tracer is not ddtrace.tracer:
+            deprecate(
+                "Initializing ddtrace.trace.Pin with `tracer` argument is deprecated",
+                message="All Pin instances should use the global tracer instance",
+                removal_version="3.0.0",
+            )
+        tracer = tracer or ddtrace.tracer
         self.tags = tags
-        self._tracer = ddtrace.tracer
+        self.tracer = tracer
         self._target = None  # type: Optional[int]
         # keep the configuration attribute internal because the
         # public API to access it is not the Pin class
@@ -59,14 +68,10 @@ def service(self):
         return self._config["service_name"]
 
     def __setattr__(self, name, value):
-        if getattr(self, "_initialized", False) and name not in ("_target", "_tracer"):
+        if getattr(self, "_initialized", False) and name != "_target":
             raise AttributeError("can't mutate a pin, use override() or clone() instead")
         super(Pin, self).__setattr__(name, value)
 
-    @property
-    def tracer(self):
-        return self._tracer
-
     def __repr__(self):
         return "Pin(service=%s, tags=%s, tracer=%s)" % (self.service, self.tags, self.tracer)
 
@@ -122,6 +127,7 @@ def override(
         obj,  # type: Any
         service=None,  # type: Optional[str]
         tags=None,  # type: Optional[Dict[str, str]]
+        tracer=None,
     ):
         # type: (...) -> None
         """Override an object with the given attributes.
@@ -133,32 +139,20 @@ def override(
             >>> # Override a pin for a specific connection
             >>> Pin.override(conn, service='user-db')
         """
-        Pin._override(obj, service=service, tags=tags)
-
-    @classmethod
-    def _override(
-        cls,
-        obj,  # type: Any
-        service=None,  # type: Optional[str]
-        tags=None,  # type: Optional[Dict[str, str]]
-        tracer=None,
-    ):
-        # type: (...) -> None
-        """
-        Internal method that allows overriding the global tracer in tests
-        """
+        if tracer is not None:
+            deprecate(
+                "Calling ddtrace.trace.Pin.override(...) with the `tracer` argument is deprecated",
+                message="All Pin instances should use the global tracer instance",
+                removal_version="3.0.0",
+            )
         if not obj:
             return
 
         pin = cls.get_from(obj)
         if pin is None:
-            pin = Pin(service=service, tags=tags)
+            Pin(service=service, tags=tags, tracer=tracer).onto(obj)
         else:
-            pin = pin.clone(service=service, tags=tags)
-
-        if tracer:
-            pin._tracer = tracer
-        pin.onto(obj)
+            pin.clone(service=service, tags=tags, tracer=tracer).onto(obj)
 
     def enabled(self):
         # type: () -> bool
@@ -204,22 +198,21 @@ def clone(
         self,
         service=None,  # type: Optional[str]
         tags=None,  # type: Optional[Dict[str, str]]
+        tracer=None,
     ):
         # type: (...) -> Pin
         """Return a clone of the pin with the given attributes replaced."""
-        return self._clone(service=service, tags=tags)
-
-    def _clone(
-        self,
-        service=None,  # type: Optional[str]
-        tags=None,  # type: Optional[Dict[str, str]]
-        tracer=None,
-    ):
-        """Internal method that can clone the tracer from an existing Pin. This is used in tests"""
         # do a shallow copy of Pin dicts
         if not tags and self.tags:
             tags = self.tags.copy()
 
+        if tracer is not None:
+            deprecate(
+                "Initializing ddtrace.trace.Pin with `tracer` argument is deprecated",
+                message="All Pin instances should use the global tracer instance",
+                removal_version="3.0.0",
+            )
+
         # we use a copy instead of a deepcopy because we expect configurations
         # to have only a root level dictionary without nested objects. Using
         # deepcopy introduces a big overhead:
@@ -228,10 +221,9 @@ def _clone(
         # deepcopy: 0.2787208557128906
         config = self._config.copy()
 
-        pin = Pin(
+        return Pin(
             service=service or self.service,
             tags=tags,
+            tracer=tracer or self.tracer,  # do not clone the Tracer
             _config=config,
         )
-        pin._tracer = tracer or self.tracer
-        return pin

ddtrace/contrib/internal/asgi/middleware.py

@@ -150,8 +150,12 @@ async def __call__(self, scope, receive, send):
         if scope["type"] == "http":
             operation_name = schematize_url_operation(operation_name, direction=SpanDirection.INBOUND, protocol="http")
 
-        pin = ddtrace.trace.Pin(service="asgi")
-        pin._tracer = self.tracer
+        # Calling ddtrace.trace.Pin(...) with the `tracer` argument is deprecated
+        # Remove this if statement when the `tracer` argument is removed
+        if self.tracer is ddtrace.tracer:
+            pin = ddtrace.trace.Pin(service="asgi")
+        else:
+            pin = ddtrace.trace.Pin(service="asgi", tracer=self.tracer)
 
         with core.context_with_data(
             "asgi.__call__",

ddtrace/contrib/internal/django/patch.py

@@ -17,6 +17,7 @@
 import wrapt
 from wrapt.importer import when_imported
 
+import ddtrace
 from ddtrace import config
 from ddtrace.appsec._utils import _UserInfoRetriever
 from ddtrace.constants import SPAN_KIND
@@ -148,9 +149,12 @@ def cursor(django, pin, func, instance, args, kwargs):
         tags = {"django.db.vendor": vendor, "django.db.alias": alias}
         tags.update(getattr(conn, "_datadog_tags", {}))
 
-        tracer = pin.tracer
-        pin = Pin(service, tags=tags)
-        pin._tracer = tracer
+        # Calling ddtrace.pin.Pin(...) with the `tracer` argument generates a deprecation warning.
+        # Remove this if statement when the `tracer` argument is removed
+        if pin.tracer is ddtrace.tracer:
+            pin = Pin(service, tags=tags)
+        else:
+            pin = Pin(service, tags=tags, tracer=pin.tracer)
 
         cursor = func(*args, **kwargs)
 

ddtrace/contrib/internal/mongoengine/trace.py

@@ -29,10 +29,11 @@ def __call__(self, *args, **kwargs):
         client = self.__wrapped__(*args, **kwargs)
         pin = ddtrace.trace.Pin.get_from(self)
         if pin:
-            tracer = pin.tracer
-            pp = ddtrace.trace.Pin(service=pin.service)
-            if tracer is not None:
-                pp._tracer = tracer
-            pp.onto(client)
+            # Calling ddtrace.trace.Pin(...) with the `tracer` argument generates a deprecation warning.
+            # Remove this if statement when the `tracer` argument is removed
+            if pin.tracer is ddtrace.tracer:
+                ddtrace.trace.Pin(service=pin.service).onto(client)
+            else:
+                ddtrace.trace.Pin(service=pin.service, tracer=pin.tracer).onto(client)
 
         return client

ddtrace/contrib/internal/pylibmc/client.py

@@ -51,8 +51,12 @@ def __init__(self, client=None, service=memcached.SERVICE, tracer=None, *args, *
         super(TracedClient, self).__init__(client)
 
         schematized_service = schematize_service_name(service)
-        pin = ddtrace.trace.Pin(service=schematized_service)
-        pin._tracer = tracer
+        # Calling ddtrace.trace.Pin(...) with the `tracer` argument generates a deprecation warning.
+        # Remove this if statement when the `tracer` argument is removed
+        if tracer is ddtrace.tracer:
+            pin = ddtrace.trace.Pin(service=schematized_service)
+        else:
+            pin = ddtrace.trace.Pin(service=schematized_service, tracer=tracer)
         pin.onto(self)
 
         # attempt to collect the pool of urls this client talks to

ddtrace/contrib/internal/sqlalchemy/engine.py

@@ -67,9 +67,12 @@ def __init__(self, tracer, service, engine):
         self.name = schematize_database_operation("%s.query" % self.vendor, database_provider=self.vendor)
 
         # attach the PIN
-        pin = Pin(service=self.service)
-        pin._tracer = self.tracer
-        pin.onto(engine)
+        # Calling ddtrace.trace.Pin(...) with the `tracer` argument generates a deprecation warning.
+        # Remove this if statement when the `tracer` argument is removed
+        if self.tracer is ddtrace.tracer:
+            Pin(service=self.service).onto(engine)
+        else:
+            Pin(tracer=tracer, service=self.service).onto(engine)
 
         listen(engine, "before_cursor_execute", self._before_cur_exec)
         listen(engine, "after_cursor_execute", self._after_cur_exec)

ddtrace/contrib/internal/tornado/application.py

@@ -54,6 +54,10 @@ def tracer_config(__init__, app, args, kwargs):
     if tags:
         tracer.set_tags(tags)
 
-    pin = ddtrace.trace.Pin(service=service)
-    pin._tracer = tracer
-    pin.onto(template)
+    # configure the PIN object for template rendering
+    # Required for backwards compatibility. Remove the else clause when
+    # the `ddtrace.trace.Pin` object no longer accepts the Pin argument.
+    if tracer is ddtrace.tracer:
+        ddtrace.trace.Pin(service=service).onto(template)
+    else:
+        ddtrace.trace.Pin(service=service, tracer=tracer).onto(template)

releasenotes/notes/remove-deprecations-from-pin-a700ae0cb6d51d03.yaml

@@ -196,7 +196,7 @@ def login_user(request):
 def new_service(request, service_name: str):
     import ddtrace
 
-    ddtrace.trace.Pin._override(django, service=service_name, tracer=ddtrace.tracer)
+    ddtrace.trace.Pin.override(django, service=service_name, tracer=ddtrace.tracer)
     return HttpResponse(service_name, status=200)
 
 

tests/appsec/contrib_appsec/django_app/urls.py

@@ -104,7 +104,7 @@ async def multi_view_no_param(request: Request):  # noqa: B008
     async def new_service(service_name: str, request: Request):  # noqa: B008
         import ddtrace
 
-        ddtrace.trace.Pin._override(app, service=service_name, tracer=ddtrace.tracer)
+        ddtrace.trace.Pin.override(app, service=service_name, tracer=ddtrace.tracer)
         return HTMLResponse(service_name, 200)
 
     async def slow_numbers(minimum, maximum):

tests/appsec/contrib_appsec/fastapi_app/app.py

@@ -55,7 +55,7 @@ def multi_view(param_int=0, param_str=""):
 def new_service(service_name: str):
     import ddtrace
 
-    ddtrace.trace.Pin._override(Flask, service=service_name, tracer=ddtrace.tracer)
+    ddtrace.trace.Pin.override(Flask, service=service_name, tracer=ddtrace.tracer)
     return service_name
 
 

tests/appsec/contrib_appsec/flask_app/app.py

@@ -37,7 +37,7 @@ def setUp(self):
         self.app = app
         self.app.test_client_class = DDFlaskTestClient
         self.client = self.app.test_client()
-        Pin._override(self.app, tracer=self.tracer)
+        Pin.override(self.app, tracer=self.tracer)
 
     def tearDown(self):
         super(BaseFlaskTestCase, self).tearDown()

tests/appsec/contrib_appsec/test_flask.py

@@ -1568,7 +1568,7 @@ def test_tracer():
 @contextmanager
 def post_tracer(interface):
     original_tracer = getattr(ddtrace.trace.Pin.get_from(interface.framework), "tracer", None)
-    ddtrace.trace.Pin._override(interface.framework, tracer=interface.tracer)
+    ddtrace.trace.Pin.override(interface.framework, tracer=interface.tracer)
     yield
     if original_tracer is not None:
-        ddtrace.trace.Pin._override(interface.framework, tracer=original_tracer)
+        ddtrace.trace.Pin.override(interface.framework, tracer=original_tracer)

tests/appsec/contrib_appsec/utils.py

@@ -50,7 +50,7 @@ def tracer():
     # Patch Django and override tracer to be our test tracer
     pin = Pin.get_from(django)
     original_tracer = pin.tracer
-    Pin._override(django, tracer=tracer)
+    Pin.override(django, tracer=tracer)
 
     # Yield to our test
     yield tracer
@@ -59,7 +59,7 @@ def tracer():
     # Reset the tracer pinned to Django and unpatch
     # DEV: unable to properly unpatch and reload django app with each test
     # unpatch()
-    Pin._override(django, tracer=original_tracer)
+    Pin.override(django, tracer=original_tracer)
 
 
 @pytest.fixture

tests/appsec/integrations/django_tests/conftest.py

@@ -42,11 +42,11 @@ async def aiobotocore_client(service, tracer):
         client, aiobotocore.session.ClientCreatorContext
     ):
         async with client as client:
-            Pin._override(client, tracer=tracer)
+            Pin.override(client, tracer=tracer)
             await yield_(client)
 
     else:
-        Pin._override(client, tracer=tracer)
+        Pin.override(client, tracer=tracer)
         try:
             await yield_(client)
         finally:

tests/contrib/aiobotocore/utils.py

@@ -189,7 +189,7 @@ def test_configure_service_name_pin(ddtrace_run_python_code_in_subprocess):
 
 async def test():
     async with aiohttp.ClientSession() as session:
-        Pin._override(session, service="pin-custom-svc")
+        Pin.override(session, service="pin-custom-svc")
         async with session.get(URL_200) as resp:
             pass
 

tests/contrib/aiohttp/test_aiohttp_client.py

@@ -13,7 +13,7 @@
 def patched_app_tracer_jinja(patched_app_tracer):  # noqa: F811
     app, tracer = patched_app_tracer
     patch()
-    Pin._override(aiohttp_jinja2, tracer=tracer)
+    Pin.override(aiohttp_jinja2, tracer=tracer)
     yield app, tracer
     unpatch()
 
@@ -22,6 +22,6 @@ def patched_app_tracer_jinja(patched_app_tracer):  # noqa: F811
 def untraced_app_tracer_jinja(untraced_app_tracer):  # noqa: F811
     patch()
     app, tracer = untraced_app_tracer
-    Pin._override(aiohttp_jinja2, tracer=tracer)
+    Pin.override(aiohttp_jinja2, tracer=tracer)
     yield app, tracer
     unpatch()

tests/contrib/aiohttp_jinja2/conftest.py

@@ -35,7 +35,7 @@ async def test_template_rendering(untraced_app_tracer_jinja, aiohttp_client):
 
 async def test_template_rendering_snapshot(untraced_app_tracer_jinja, aiohttp_client, snapshot_context):
     app, _ = untraced_app_tracer_jinja
-    Pin._override(aiohttp_jinja2, tracer=tracer)
+    Pin.override(aiohttp_jinja2, tracer=tracer)
     with snapshot_context():
         client = await aiohttp_client(app)
         # it should trace a template rendering
@@ -51,7 +51,7 @@ async def test_template_rendering_snapshot_patched_server(
     use_global_tracer,
 ):
     app, _ = patched_app_tracer_jinja
-    Pin._override(aiohttp_jinja2, tracer=tracer)
+    Pin.override(aiohttp_jinja2, tracer=tracer)
     # Ignore meta.http.url tag as the port is not fixed on the server
     with snapshot_context(ignores=["meta.http.url", "meta.http.useragent"]):
         client = await aiohttp_client(app)