Island: Make FileNotFoundError inherit UnknownRecordError instead of RetrievalError + add TODO for updating the rest of the code

This commit is contained in:
Shreya Malviya 2022-09-28 19:55:52 +05:30
parent 64990eea0e
commit 8f46b3b9fd
2 changed files with 4 additions and 11 deletions

View File

@ -1,14 +1,8 @@
import io
import re
from monkey_island.cc import repository
from monkey_island.cc.models import AgentID
from monkey_island.cc.repository import (
IAgentLogRepository,
IFileRepository,
RetrievalError,
UnknownRecordError,
)
from monkey_island.cc.repository import IAgentLogRepository, IFileRepository, RetrievalError
AGENT_LOG_FILE_NAME_PATTERN = "agent-*.log"
AGENT_LOG_FILE_NAME_REGEX = re.compile(r"^agent-[\w-]+.log$")
@ -28,8 +22,6 @@ class FileAgentLogRepository(IAgentLogRepository):
with self._file_repository.open_file(self._get_agent_log_file_name(agent_id)) as f:
log_contents = f.read().decode()
return log_contents
except repository.FileNotFoundError as err:
raise UnknownRecordError(err)
except Exception as err:
raise RetrievalError(f"Error retrieving the agent logs: {err}")

View File

@ -1,10 +1,11 @@
import abc
from typing import BinaryIO
from monkey_island.cc.repository import RetrievalError
from monkey_island.cc.repository import UnknownRecordError
class FileNotFoundError(RetrievalError):
# TODO: Remove this and use UnknownRecordError directly wherever needed.
class FileNotFoundError(UnknownRecordError):
pass