keryx 0.12.2 → 0.12.4

package/classes/Connection.ts

@@ -100,6 +100,7 @@ export class Connection<
     let error: TypedError | undefined;
 
     let action: Action | undefined;
+    let formattedParams: Record<string, unknown> | undefined;
     try {
       action = this.findAction(actionName);
       if (!action) {
@@ -112,7 +113,7 @@ export class Connection<
       // load the session once, if it hasn't been loaded yet
       if (!this.sessionLoaded) await this.loadSession();
 
-      let formattedParams = await this.formatParams(params, action);
+      formattedParams = await this.formatParams(params, action);
 
       for (const middleware of action.middleware ?? []) {
         if (middleware.runBefore) {
@@ -145,17 +146,6 @@ export class Connection<
       } else {
         response = await action.run(formattedParams, this);
       }
-
-      for (const middleware of action.middleware ?? []) {
-        if (middleware.runAfter) {
-          const middlewareResponse = await middleware.runAfter(
-            formattedParams,
-            this,
-          );
-          if (middlewareResponse && middlewareResponse?.updatedResponse)
-            response = middlewareResponse.updatedResponse;
-        }
-      }
     } catch (e) {
       loggerResponsePrefix = "ERROR";
       error =
@@ -166,6 +156,19 @@ export class Connection<
               type: ErrorType.CONNECTION_ACTION_RUN,
               originalError: e,
             });
+    } finally {
+      if (action && formattedParams) {
+        for (const middleware of action.middleware ?? []) {
+          if (middleware.runAfter) {
+            const middlewareResponse = await middleware.runAfter(
+              formattedParams,
+              this,
+            );
+            if (middlewareResponse && middlewareResponse?.updatedResponse)
+              response = middlewareResponse.updatedResponse;
+          }
+        }
+      }
     }
 
     const duration = new Date().getTime() - reqStartTime;
@@ -410,9 +413,20 @@ const sanitizeParams = (
     }
   }
 
+  const maxParamLength = config.logger.maxParamLength;
+
   for (const [k, v] of Object.entries(params)) {
     if (secretFields.has(k)) {
       sanitizedParams[k] = REDACTED;
+    } else if (maxParamLength > 0) {
+      const stringified = typeof v === "string" ? v : JSON.stringify(v);
+      if (stringified && stringified.length > maxParamLength) {
+        sanitizedParams[k] =
+          stringified.slice(0, maxParamLength) +
+          `... (truncated, original length: ${stringified.length})`;
+      } else {
+        sanitizedParams[k] = v;
+      }
     } else {
       sanitizedParams[k] = v;
     }

package/config/logger.ts

@@ -6,4 +6,6 @@ export const configLogger = {
   includeTimestamps: await loadFromEnvIfSet("LOG_INCLUDE_TIMESTAMPS", true),
   colorize: await loadFromEnvIfSet("LOG_COLORIZE", true),
   format: await loadFromEnvIfSet<LogFormat>("LOG_FORMAT", LogFormat.text),
+  /** Maximum length of individual param values in action logs before truncation. Set to 0 to disable truncation. */
+  maxParamLength: await loadFromEnvIfSet("LOG_MAX_PARAM_LENGTH", 100),
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "keryx",
-  "version": "0.12.2",
+  "version": "0.12.4",
   "module": "index.ts",
   "type": "module",
   "license": "MIT",