fix: pr comments

defi-wonderland · Nov 8, 2024 · 4c2d2f1 · 4c2d2f1
1 parent 71631ea
commit 4c2d2f1
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/packages/automated-dispute/src/providers/protocolProvider.ts b/packages/automated-dispute/src/providers/protocolProvider.ts
@@ -257,7 +257,7 @@ export class ProtocolProvider implements IProtocolProvider {
                     url: request.url,
                     method: request.method,
                     headers: Array.from(request.headers.entries()),
-                    body: await request.text(),
+                    body: await request.clone().text(),
                 };
 
                 this.logger.debug(`Fetch Request`, reqObject);
@@ -267,7 +267,7 @@ export class ProtocolProvider implements IProtocolProvider {
                     url: response.url,
                     status: response.status,
                     headers: Array.from(response.headers.entries()),
-                    body: await response.text(),
+                    body: await response.clone().text(),
                 };
 
                 this.logger.debug(`Fetch Response`, respObject);

diff --git a/packages/shared/src/services/logger.ts b/packages/shared/src/services/logger.ts
@@ -5,8 +5,21 @@ import { ILogger } from "../index.js";
 
 type LogLevel = "error" | "warn" | "info" | "debug";
 
+type LogEntry = {
+    timestamp: string;
+    level: LogLevel;
+    message: string;
+    stack?: string;
+    data?: Record<string, unknown>;
+};
+
 const validLogLevels: LogLevel[] = ["error", "warn", "info", "debug"];
 
+/**
+ * Logger class that wraps the Winston logger.
+ * @class Logger
+ * @implements {ILogger}
+ */
 export class Logger implements ILogger {
     private logger: WinstonLogger;
     private static instance: Logger | null;
@@ -21,9 +34,9 @@ export class Logger implements ILogger {
                 format.errors({ stack: true }),
                 format((info) => {
                     const { timestamp, level, message, stack, ...meta } = info;
-                    const logEntry: any = {
+                    const logEntry: LogEntry = {
                         timestamp,
-                        level,
+                        level: this.isValidLogLevel(level) ? level : "info",
                         message,
                     };
                     if (stack) {
@@ -51,8 +64,8 @@ export class Logger implements ILogger {
         return Logger.instance;
     }
 
-    isValidLogLevel(level: any): level is LogLevel {
-        return validLogLevels.includes(level);
+    isValidLogLevel(level: string | undefined): level is LogLevel {
+        return validLogLevels.includes(level as LogLevel);
     }
 
     info(message: string, meta?: unknown) {