refactor(fastapi): drop __del__ as WeakSet handles cleanup

artemziborev · artemziborev · commit a0546d48f91e · 2025-08-12T17:12:02.000+04:00
diff --git a/instrumentation/opentelemetry-instrumentation-fastapi/src/opentelemetry/instrumentation/fastapi/__init__.py b/instrumentation/opentelemetry-instrumentation-fastapi/src/opentelemetry/instrumentation/fastapi/__init__.py
@@ -437,10 +437,6 @@ def __init__(self, *args, **kwargs):
         )
         _InstrumentedFastAPI._instrumented_fastapi_apps.add(self)
 
-    def __del__(self):
-        # Best-effort cleanup; WeakSet clears references on GC automatically
-        _InstrumentedFastAPI._instrumented_fastapi_apps.discard(self)
-
 
 def _get_route_details(scope):
     """

Original file line number	Diff line number	Diff line change
`@@ -437,10 +437,6 @@ def __init__(self, args, *kwargs):`
`437`	`437`	`)`
`438`	`438`	`_InstrumentedFastAPI._instrumented_fastapi_apps.add(self)`
`439`	`439`
`440`		`- def __del__(self):`
`441`		`- # Best-effort cleanup; WeakSet clears references on GC automatically`
`442`		`- _InstrumentedFastAPI._instrumented_fastapi_apps.discard(self)`
`443`		`-`
`444`	`440`
`445`	`441`	`def _get_route_details(scope):`
`446`	`442`	`"""`