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`);