@envoy/envoy-integrations-sdk 1.3.2 → 1.3.3

package/lib/axios.js

@@ -8,12 +8,15 @@ function createAxiosClient(config) {
            return response 
         },
         (error) => {
-            // if the response class type is AxiosError, scrub config fields containing sensitive data
-            if (error instanceof AxiosError) {
-                delete error.config.headers;
-                delete error.config.proxy;
+            const safeError = {
+                code: error.code,
+                message: error.message,
+                name: error.name,
+                baseURL: error.request?.baseURL ?? error.config?.baseURL,
+                url: error.request?.url ?? error.config?.url,
+                method: error.request?.method ?? error.config?.method,
             }
-            return Promise.reject(error);
+            return Promise.reject(safeError);
         },
     );
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@envoy/envoy-integrations-sdk",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "",
   "main": "index.js",
   "scripts": {

package/test/axios.test.js

@@ -1,5 +1,5 @@
 const axios = require('axios');
-const createClient = require('../lib/axios')
+const createClient = require('../lib/axios');
 
 describe('axios', () => {
     it('default client should leak credentials in AxiosError', async () => {
@@ -18,8 +18,7 @@ describe('axios', () => {
         try {
             await client.get('http://localhost:3000/axios-error');
         } catch (error) {
-            expect(error.config.headers).toBeDefined();
-            expect(error.config.proxy).toBeDefined();
+            expect(error.config).toBeDefined();
             const errorStr = JSON.stringify(error);
             expect(errorStr).toContain('Bearer 1234');
             expect(errorStr).toContain('myWhackyUsername');
@@ -43,8 +42,12 @@ describe('axios', () => {
         try {
             await client.get('http://localhost:3000/axios-error');
         } catch (error) {
-            expect(error.config.headers).toBeUndefined();
-            expect(error.config.proxy).toBeUndefined();
+            expect(error.config).toBeUndefined();
+            expect(error.code).toBe('ECONNREFUSED');
+            expect(error.message).toBe('connect ECONNREFUSED 127.0.0.1:80');
+            expect(error.name).toBe('Error');
+            expect(error.url).toBe('http://localhost:3000/axios-error');
+            expect(error.method).toBe('get');
             const errorStr = JSON.stringify(error);
             expect(errorStr).not.toContain('Bearer 1234');
             expect(errorStr).not.toContain('myWhackyUsername');