Skip to content

Commit 876adf8

Browse files
committed
fix: update serializeLogData function implementation
1 parent bf9b00e commit 876adf8

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/libs/Middleware/Logging.ts

+7-2
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,13 @@ function getCircularReplacer() {
2525
};
2626
}
2727

28-
function serializeLoggingData(logData?: Record<string, unknown> | null): Record<string, unknown> | undefined | null {
29-
return logData && (JSON.parse(JSON.stringify(logData, getCircularReplacer())) as Record<string, unknown>);
28+
function serializeLoggingData<T extends Record<string, unknown> | undefined>(logData: T): T | null {
29+
try {
30+
return JSON.parse(JSON.stringify(logData, getCircularReplacer())) as T;
31+
} catch (error) {
32+
Log.hmmm('Failed to serialize log data', {error});
33+
return null;
34+
}
3035
}
3136

3237
function logRequestDetails(message: string, request: Request, response?: Response | void) {

0 commit comments

Comments
 (0)