Skip to content

Commit 966b9ce

Browse files
authored
Log: syslog formatting fixes (#12324)
This broke on production because `self._level_to_color` and `self._pad_event` don't exist anymore in the class. We will need to migrate this code if we want.
1 parent a359d48 commit 966b9ce

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

readthedocs/core/logs.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,8 @@ def __call__(self, logger, method_name, event_dict):
8282

8383

8484
class SysLogRenderer(structlog.dev.ConsoleRenderer):
85+
# Line generated by our custom ``SysLogRenderer``
86+
# %programname[%processid]: %timestamp [%level] %message [%logger:%linenumber] %structureddata
8587
def __call__(self, logger, name, event_dict):
8688
sio = StringIO()
8789

@@ -108,9 +110,12 @@ def __call__(self, logger, name, event_dict):
108110
if level is not None:
109111
sio.write(
110112
"["
111-
+ self._level_to_color.get(level, "")
113+
# NOTE: I'm removing the color for now to avoid failing.
114+
# we could adapt this code to work with the latest version of structlog.
115+
#
116+
# + self._level_to_color.get(level, "")
112117
+ _pad(level, self._longest_level)
113-
+ self._styles.reset
118+
# + self._styles.reset
114119
+ "] "
115120
)
116121

@@ -120,7 +125,7 @@ def __call__(self, logger, name, event_dict):
120125
event = str(event)
121126

122127
if event_dict:
123-
event = _pad(event, self._pad_event) + self._styles.reset + " "
128+
event = _pad(event, 50) + self._styles.reset + " "
124129
else:
125130
event += self._styles.reset
126131
sio.write(self._styles.bright + event)

0 commit comments

Comments
 (0)