From 0dda40677f40274d26d561601eb67a5413c2e3cc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Jerna=C5=9B?= Date: Tue, 6 Feb 2024 15:50:29 +0100 Subject: [PATCH] Log multiple errors # What Log AggregateError type, when multiple errors are returned from HTTP client - fixes #174 # Why We would silently fail otherwise as error.message was empty for the AggregatedError exception. --- dist/index.js | 8 +++++++- src/main.ts | 7 ++++++- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dist/index.js b/dist/index.js index 66c90a7..c30de4f 100644 --- a/dist/index.js +++ b/dist/index.js @@ -28050,7 +28050,13 @@ async function run() { } } catch (error) { - if (error instanceof Error) { + if (error instanceof AggregateError) { + core.setFailed(`Multiple errors returned`); + for (const err of error.errors) { + core.error(`Error ${error.errors.indexOf(err)}: ${err.message}`); + } + } + else if (error instanceof Error) { core.setFailed(error.message); } else { diff --git a/src/main.ts b/src/main.ts index 98096e4..041a3d4 100644 --- a/src/main.ts +++ b/src/main.ts @@ -25,7 +25,12 @@ export async function run(): Promise { } } } catch (error) { - if (error instanceof Error) { + if (error instanceof AggregateError) { + core.setFailed(`Multiple errors returned`) + for (const err of error.errors) { + core.error(`Error ${error.errors.indexOf(err)}: ${err.message}`) + } + } else if (error instanceof Error) { core.setFailed(error.message) } else { core.setFailed(`Unknown object was thrown: ${error}`)