Skip to content

Commit 844c5e0

Browse files
committed
fix: unlink error with some closed tempfiles
1 parent a6a5f2b commit 844c5e0

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

src/modules/printing/repository.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,10 @@ async def wait_for_tempfile_expiration(self, innohassle_user_id, f):
6262
def remove_tempfile(self, innohassle_user_id, filename, expired=False):
6363
if (innohassle_user_id, filename) in self.tempfiles:
6464
self.tempfiles[(innohassle_user_id, filename)][0].close()
65-
os.unlink(self.get_tempfile_path(innohassle_user_id, filename))
65+
try:
66+
os.unlink(self.get_tempfile_path(innohassle_user_id, filename))
67+
except FileNotFoundError:
68+
pass
6669
if not expired:
6770
self.tempfiles[(innohassle_user_id, filename)][1].cancel()
6871
del self.tempfiles[(innohassle_user_id, filename)]

src/modules/scanning/repository.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,10 @@ def retrieve_tempfile(self, innohassle_user_id: USER_AUTH, filename: str):
8080
def remove_tempfile(self, innohassle_user_id: USER_AUTH, filename: str, expired=False):
8181
if (innohassle_user_id, filename) in self.tempfiles:
8282
self.tempfiles[(innohassle_user_id, filename)][0].close()
83-
os.unlink(self.get_tempfile_path(innohassle_user_id, filename))
83+
try:
84+
os.unlink(self.get_tempfile_path(innohassle_user_id, filename))
85+
except FileNotFoundError:
86+
pass
8487
if not expired:
8588
self.tempfiles[(innohassle_user_id, filename)][1].cancel()
8689
del self.tempfiles[(innohassle_user_id, filename)]

0 commit comments

Comments
 (0)