remove lock

bentsku · bentsku · commit 302a4fcac393 · 2025-08-19T17:51:36.000+02:00
diff --git a/localstack-core/localstack/utils/patch.py b/localstack-core/localstack/utils/patch.py
@@ -1,6 +1,5 @@
 import functools
 import inspect
-import threading
 import types
 from typing import Any, Callable
 
@@ -73,7 +72,6 @@ def proxy(*args, **kwargs):
 
 class Patch:
     applied_patches: list["Patch"] = []
-    _lock: threading.RLock = threading.RLock()
     """Bookkeeping for patches that are applied. You can use this to debug patches. For instance,
     you could write something like::
 
@@ -99,27 +97,25 @@ def __init__(self, obj: Any, name: str, new: Any) -> None:
         self.is_applied = False
 
     def apply(self):
-        with self._lock:
-            if self.is_applied:
-                return
-
-            if self.old and self.name == "__getattr__":
-                raise Exception("You can't patch class types implementing __getattr__")
-            if not self.old and self.name != "__getattr__":
-                raise AttributeError(f"`{self.obj.__name__}` object has no attribute `{self.name}`")
-            setattr(self.obj, self.name, self.new)
-            Patch.applied_patches.append(self)
-            self.is_applied = True
+        if self.is_applied:
+            return
+
+        if self.old and self.name == "__getattr__":
+            raise Exception("You can't patch class types implementing __getattr__")
+        if not self.old and self.name != "__getattr__":
+            raise AttributeError(f"`{self.obj.__name__}` object has no attribute `{self.name}`")
+        setattr(self.obj, self.name, self.new)
+        Patch.applied_patches.append(self)
+        self.is_applied = True
 
     def undo(self):
-        with self._lock:
-            if not self.is_applied:
-                return
-
-            # If we added a method to a class type, we don't have a self.old. We just delete __getattr__
-            setattr(self.obj, self.name, self.old) if self.old else delattr(self.obj, self.name)
-            Patch.applied_patches.remove(self)
-            self.is_applied = False
+        if not self.is_applied:
+            return
+
+        # If we added a method to a class type, we don't have a self.old. We just delete __getattr__
+        setattr(self.obj, self.name, self.old) if self.old else delattr(self.obj, self.name)
+        Patch.applied_patches.remove(self)
+        self.is_applied = False
 
     def __enter__(self):
         self.apply()
