@vtex/api 6.45.6 → 6.45.7

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 ## [Unreleased]
 
+## [6.45.7] - 2022-02-23
+
+### Fixed
+
+- Remove sensitive data from logs.
+
 ## [6.45.6] - 2021-10-26
 
 ### Fixed

package/lib/service/logger/logger.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const constants_1 = require("../../constants");
 const error_1 = require("../../utils/error");
+const log_1 = require("../../utils/log");
 const console_1 = require("./console");
 const linked = !!process.env.VTEX_APP_LINK;
 const app = constants_1.APP.ID;
@@ -22,6 +23,7 @@ class Logger {
         this.log = (message, level) => {
             var _a;
             const data = message ? error_1.cleanError(message) : EMPTY_MESSAGE;
+            log_1.cleanLog(data);
             /* tslint:disable:object-literal-sort-keys */
             const inflatedLog = {
                 __VTEX_IO_LOG: true,

package/lib/service/worker/runtime/graphql/utils/error.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const ramda_1 = require("ramda");
 const error_1 = require("../../../../../utils/error");
+const log_1 = require("../../../../../utils/log");
 const ERROR_FIELD_WHITELIST = ['message', 'path', 'stack', 'extensions', 'statusCode', 'name', 'headers', 'originalError', 'code'];
 const QUERY_FIELDS = ['query', 'operationName', 'variables'];
 const trimVariables = (variables) => {
@@ -48,8 +49,8 @@ const formatError = (error, details) => {
             formattedError.extensions.exception = {
                 message: formattedError.originalError.message,
                 name: formattedError.originalError.name,
-                ...ramda_1.omit(error_1.SENSITIVE_EXCEPTION_FIELDS, formattedError.originalError),
-                sensitive: ramda_1.pick(error_1.SENSITIVE_EXCEPTION_FIELDS, formattedError.originalError),
+                ...ramda_1.omit(log_1.FIRST_LEVEL_SENSITIVE_FIELDS, formattedError.originalError),
+                sensitive: ramda_1.pick(log_1.FIRST_LEVEL_SENSITIVE_FIELDS, formattedError.originalError),
             };
         }
         else {
@@ -59,8 +60,8 @@ const formatError = (error, details) => {
             const extendedException = {
                 message: formattedError.originalError.message,
                 name: formattedError.originalError.name,
-                ...ramda_1.omit(error_1.SENSITIVE_EXCEPTION_FIELDS, mergedException),
-                sensitive: ramda_1.pick(error_1.SENSITIVE_EXCEPTION_FIELDS, mergedException),
+                ...ramda_1.omit(log_1.FIRST_LEVEL_SENSITIVE_FIELDS, mergedException),
+                sensitive: ramda_1.pick(log_1.FIRST_LEVEL_SENSITIVE_FIELDS, mergedException),
             };
             formattedError.extensions.exception = error_1.cleanError(extendedException);
         }

package/lib/service/worker/runtime/http/middlewares/error.js

@@ -4,6 +4,7 @@ const constants_1 = require("../../../../../constants");
 const RequestCancelledError_1 = require("../../../../../errors/RequestCancelledError");
 const TooManyRequestsError_1 = require("../../../../../errors/TooManyRequestsError");
 const error_1 = require("../../../../../utils/error");
+const log_1 = require("../../../../../utils/log");
 const logger_1 = require("../../../../logger");
 const CACHE_CONTROL_HEADER = 'cache-control';
 const META_HEADER = 'x-vtex-meta';
@@ -39,7 +40,7 @@ const logAndRemoveSensitiveData = (ctx, err) => {
     // Use sendLog directly to avoid cleaning error twice.
     ctx.vtex.logger.log(log, level);
     if (!constants_1.LINKED) {
-        error_1.SENSITIVE_EXCEPTION_FIELDS.forEach(field => {
+        log_1.FIRST_LEVEL_SENSITIVE_FIELDS.forEach(field => {
             delete err[field];
         });
     }

package/lib/utils/error.d.ts

@@ -1,5 +1,4 @@
 export declare const PICKED_AXIOS_PROPS: string[];
-export declare const SENSITIVE_EXCEPTION_FIELDS: string[];
 /**
  * Cleans errors by removing circular properties, truncating large strings and picking axios properties
  *

package/lib/utils/error.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 // Inspired by https://github.com/sindresorhus/serialize-error
 const ramda_1 = require("ramda");
 exports.PICKED_AXIOS_PROPS = ['baseURL', 'cacheable', 'data', 'finished', 'headers', 'method', 'timeout', 'status', 'path', 'url', 'metric', 'inflightKey', 'forceMaxAge', 'params', 'responseType'];
-exports.SENSITIVE_EXCEPTION_FIELDS = ['config', 'request', 'stack'];
 const MAX_ERROR_STRING_LENGTH = process.env.MAX_ERROR_STRING_LENGTH ? parseInt(process.env.MAX_ERROR_STRING_LENGTH, 10) : 8 * 1024;
 const findCaseInsensitive = (target, set) => ramda_1.find(t => t.toLocaleLowerCase() === target, set);
 const destroyCircular = (from, seen) => {

package/lib/utils/log.d.ts

@@ -0,0 +1,3 @@
+export declare const FIRST_LEVEL_SENSITIVE_FIELDS: string[];
+export declare const SECOND_LEVEL_SENSITIVE_FIELDS: string[][];
+export declare const cleanLog: (log: any) => void;

package/lib/utils/log.js

@@ -0,0 +1,14 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.FIRST_LEVEL_SENSITIVE_FIELDS = ['config', 'request', 'stack', 'error'];
+exports.SECOND_LEVEL_SENSITIVE_FIELDS = [['parsedInfo', 'requestConfig'], ['headers', 'cookie']];
+exports.cleanLog = (log) => {
+    exports.FIRST_LEVEL_SENSITIVE_FIELDS.forEach(field => {
+        delete log[field];
+    });
+    exports.SECOND_LEVEL_SENSITIVE_FIELDS.forEach(field => {
+        if (field[0] in log) {
+            delete log[field[0]][field[1]];
+        }
+    });
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vtex/api",
-  "version": "6.45.6",
+  "version": "6.45.7",
   "description": "VTEX I/O API client",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",