Use copy of state for state dumps to file

Uninett · Aug 30, 2024 · 73e016c · 73e016c
1 parent e2164d5
commit 73e016c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/changelog.d/+use-copy-for-statedump.fixed.md b/changelog.d/+use-copy-for-statedump.fixed.md
@@ -0,0 +1 @@
+Use a copy of the Zino state for saving it to a file to avoid errors due to changes to the state in the middle of saving
diff --git a/src/zino/state.py b/src/zino/state.py
@@ -42,8 +42,9 @@ class ZinoState(BaseModel):
     def dump_state_to_file(self, filename: str):
         """Dumps the full state to a file in JSON format"""
         _log.debug("dumping state to %s", filename)
+        copied_state = self.model_copy(deep=True)
         with open(filename, "w") as statefile:
-            statefile.write(self.model_dump_json(exclude_none=True, indent=2))
+            statefile.write(copied_state.model_dump_json(exclude_none=True, indent=2))
 
     @classmethod
     @log_time_spent()