refactor(main): move JSON formatter imports to module level per code review
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
+4
-5
@@ -1,8 +1,11 @@
|
|||||||
#!/usr/bin/env python3
|
#!/usr/bin/env python3
|
||||||
|
import json as _json_mod
|
||||||
import logging
|
import logging
|
||||||
import os
|
import os
|
||||||
import pathlib
|
import pathlib
|
||||||
from contextlib import asynccontextmanager
|
from contextlib import asynccontextmanager
|
||||||
|
from datetime import datetime as _dt
|
||||||
|
from datetime import timezone as _tz
|
||||||
|
|
||||||
from fastapi import FastAPI, HTTPException, Request, status
|
from fastapi import FastAPI, HTTPException, Request, status
|
||||||
from fastapi.middleware.cors import CORSMiddleware
|
from fastapi.middleware.cors import CORSMiddleware
|
||||||
@@ -75,10 +78,6 @@ class _JsonFormatter(logging.Formatter):
|
|||||||
"""
|
"""
|
||||||
|
|
||||||
def format(self, record: logging.LogRecord) -> str:
|
def format(self, record: logging.LogRecord) -> str:
|
||||||
import json as _json
|
|
||||||
from datetime import datetime as _dt
|
|
||||||
from datetime import timezone as _tz
|
|
||||||
|
|
||||||
log_entry: dict = {
|
log_entry: dict = {
|
||||||
"timestamp": _dt.fromtimestamp(record.created, tz=_tz.utc).isoformat(),
|
"timestamp": _dt.fromtimestamp(record.created, tz=_tz.utc).isoformat(),
|
||||||
"level": record.levelname,
|
"level": record.levelname,
|
||||||
@@ -90,7 +89,7 @@ class _JsonFormatter(logging.Formatter):
|
|||||||
}
|
}
|
||||||
if record.exc_info and record.exc_info[1] is not None:
|
if record.exc_info and record.exc_info[1] is not None:
|
||||||
log_entry["exc_info"] = self.formatException(record.exc_info)
|
log_entry["exc_info"] = self.formatException(record.exc_info)
|
||||||
return _json.dumps(log_entry, default=str)
|
return _json_mod.dumps(log_entry, default=str)
|
||||||
|
|
||||||
|
|
||||||
# Choose formatter based on LOG_FORMAT setting
|
# Choose formatter based on LOG_FORMAT setting
|
||||||
|
|||||||
Reference in New Issue
Block a user