From 744327c20d909a0ccc2938dff8847d2b4756d9af Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Iv=C3=A1n=20Ovejero?= Date: Wed, 3 Apr 2024 18:00:27 +0200 Subject: [PATCH] fix(core): Ensure `status` on Axios errors is available to the BE (#9015) --- packages/core/src/NodeExecuteFunctions.ts | 5 +++++ packages/core/test/NodeExecuteFunctions.test.ts | 10 ++++++++++ 2 files changed, 15 insertions(+) diff --git a/packages/core/src/NodeExecuteFunctions.ts b/packages/core/src/NodeExecuteFunctions.ts index 51f66c1574..1f61e51b23 100644 --- a/packages/core/src/NodeExecuteFunctions.ts +++ b/packages/core/src/NodeExecuteFunctions.ts @@ -848,6 +848,11 @@ export async function proxyRequestToAxios( error.message = `${response.status as number} - ${JSON.stringify(responseData)}`; throw Object.assign(error, { statusCode: response.status, + /** + * Axios adds `status` when serializing, causing `status` to be available only to the client. + * Hence we add it explicitly to allow the backend to use it when resolving expressions. + */ + status: response.status, error: responseData, response: pick(response, ['headers', 'status', 'statusText']), }); diff --git a/packages/core/test/NodeExecuteFunctions.test.ts b/packages/core/test/NodeExecuteFunctions.test.ts index 92f2378218..c9d4865b8b 100644 --- a/packages/core/test/NodeExecuteFunctions.test.ts +++ b/packages/core/test/NodeExecuteFunctions.test.ts @@ -243,6 +243,16 @@ describe('NodeExecuteFunctions', () => { hooks.executeHookFunctions.mockClear(); }); + test('should rethrow an error with `status` property', async () => { + nock(baseUrl).get('/test').reply(400); + + try { + await proxyRequestToAxios(workflow, additionalData, node, `${baseUrl}/test`); + } catch (error) { + expect(error.status).toEqual(400); + } + }); + test('should not throw if the response status is 200', async () => { nock(baseUrl).get('/test').reply(200); await proxyRequestToAxios(workflow, additionalData, node, `${baseUrl}/test`);