Merge pull request HathorNetwork#1378 from HathorNetwork/feat/nano-sha256-storage

feat(nano): change internal storage hashing to sha256

Author: jansegre

hathor/nanocontracts/storage/contract_storage.py

@@ -54,7 +54,7 @@ class AttrKey(TrieKey):
     key: bytes
 
     def __bytes__(self) -> bytes:
-        return _Tag.ATTR.value + hashlib.sha1(self.key).digest()
+        return _Tag.ATTR.value + hashlib.sha256(self.key).digest()
 
 
 @dataclass(frozen=True, slots=True)
@@ -127,7 +127,7 @@ class MetadataKey(TrieKey):
     key: bytes
 
     def __bytes__(self) -> bytes:
-        return _Tag.METADATA.value + hashlib.sha1(self.key).digest()
+        return _Tag.METADATA.value + hashlib.sha256(self.key).digest()
 
 
 _BLUEPRINT_ID_KEY = b'blueprint_id'

hathor/nanocontracts/storage/patricia_trie.py

@@ -69,7 +69,7 @@ def calculate_id(self) -> NodeId:
 
         This method assumes that all children already have their ids calculated.
         """
-        h = hashlib.sha1()
+        h = hashlib.sha256()
         h.update(self.key)
         if self.content is not None:
             h.update(self.content)

tests/nanocontracts/test_patricia_trie.py

@@ -151,7 +151,7 @@ def test_random_data(self) -> None:
         data = {}
         for v_int in range(20_000):
             v = str(v_int).encode('ascii')
-            k = hashlib.sha1(v).digest()
+            k = hashlib.sha256(v).digest()
             data[k] = v
             trie.update(k, v)
 
@@ -175,7 +175,7 @@ def test_commit(self) -> None:
         data = {}
         for v_int in range(20_000):
             v = str(v_int).encode('ascii')
-            k = hashlib.sha1(v).digest()
+            k = hashlib.sha256(v).digest()
             data[k] = v
             trie.update(k, v)
         trie.commit()