Skip to content

Commit 9aaeff1

Browse files
authored
Merge pull request #6293 from richtja/status_message_error_handling
ignore malformed partial status JSON messages
2 parents 6b54a9d + 10d90da commit 9aaeff1

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

avocado/core/status/server.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,11 @@
11
import asyncio
2+
import logging
23
import os
34

45
from avocado.core.settings import settings
6+
from avocado.core.status.utils import StatusMsgInvalidJSONError
7+
8+
LOG = logging.getLogger(__name__)
59

610

711
class StatusServer:
@@ -56,4 +60,7 @@ async def cb(self, reader, _):
5660
continue
5761
if not raw_message:
5862
return
59-
self._repo.process_raw_message(raw_message)
63+
try:
64+
self._repo.process_raw_message(raw_message)
65+
except StatusMsgInvalidJSONError as e:
66+
LOG.warning("Invalid JSON in internal status message: %s", e)

0 commit comments

Comments
 (0)