fix: checksum cache race condition

evmspec/data/_main.py

@@ -90,7 +90,6 @@ def _decode_hook(cls, typ: Type["Address"], obj: str):
         return cls.checksum(obj)
 
     @classmethod
-    @ttl_cache(maxsize=None, ttl=600)
     def checksum(cls, address: str) -> Self:
         """Returns the checksummed version of the address.
 
@@ -111,6 +110,18 @@ def checksum(cls, address: str) -> Self:
         See Also:
             - `cchecksum.to_checksum_address`: Function used for checksum conversion.
         """
+        # there is a race condition in the cache expire function due to
+        # my hacky removal of the threading.Lock from ttl_cache, but
+        # the race condition can be ignored for our use-case
+        while True:
+            try:
+                return cls.__checksum(address)
+            except KeyError:
+                pass
+
+    @classmethod
+    @ttl_cache(maxsize=None, ttl=600)
+    def __checksum(cls, address: str) -> Self:
         return cls(address)
 
 

evmspec/structs/log.py

@@ -62,15 +62,7 @@ def as_address(self) -> Address:
                 f"This {type(self).__name__} does not represent an address", self
             )
 
-        # there is a race condition in the cache expire function due to
-        # my hacky removal of the threading.Lock from ttl_cache, but
-        # the race condition can be ignored for our use-case
-        # TODO move this fix somewhere more appropriate
-        while True:
-            try:
-                return Address.checksum(self[-20:].hex())
-            except KeyError:
-                pass
+        return Address.checksum(self[-20:].hex())
 
     @property
     def as_uint8(self) -> uints.uint8: