Skip to content

Commit ed21a8b

Browse files
authored
Fix parallel aggregate error hiding real stacktrace (#164)
* fix parallel aggregate error having unhelpful stacktrace
1 parent 7aa63b7 commit ed21a8b

File tree

6 files changed

+64
-7
lines changed

6 files changed

+64
-7
lines changed

Diff for: cdn/radash.esm.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -271,8 +271,12 @@ const defer = async (func) => {
271271
return response;
272272
};
273273
class AggregateError extends Error {
274-
constructor(errors) {
274+
constructor(errors = []) {
275275
super();
276+
const name = errors.find((e) => e.name)?.name ?? "";
277+
this.name = `AggregateError(${name}...)`;
278+
this.message = `AggregateError with ${errors.length} errors`;
279+
this.stack = errors.find((e) => e.stack)?.stack ?? this.stack;
276280
this.errors = errors;
277281
}
278282
}

Diff for: cdn/radash.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -274,8 +274,12 @@ var radash = (function (exports) {
274274
return response;
275275
};
276276
class AggregateError extends Error {
277-
constructor(errors) {
277+
constructor(errors = []) {
278278
super();
279+
const name = errors.find((e) => e.name)?.name ?? "";
280+
this.name = `AggregateError(${name}...)`;
281+
this.message = `AggregateError with ${errors.length} errors`;
282+
this.stack = errors.find((e) => e.stack)?.stack ?? this.stack;
279283
this.errors = errors;
280284
}
281285
}

Diff for: cdn/radash.min.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)