yuuza/outline
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: AggregateError thrown from ValidateSSOAccessTask

Browse Source
This commit is contained in:
Tom Moor
2024-05-30 00:02:37 -04:00
parent 30c43690c0
commit 3b9cbb08c8
1 changed files with 19 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
server/queues/tasks/ValidateSSOAccessTask.ts
View File

@@ -19,17 +19,26 @@ export default class ValidateSSOAccessTask extends BaseTask<Props> {
return; return;
} }
// Check the validity of all the user's associated authentications. try {
// @ts-expect-error TODO: Need to setup nested tsconfig with ES2021 // Check the validity of all the user's associated authentications.
const valid = await Promise.any( // @ts-expect-error TODO: Need to setup nested tsconfig with ES2021
userAuthentications.map(async (authentication) => const valid = await Promise.any(
authentication.validateAccess({ transaction }) userAuthentications.map(async (authentication) =>
) authentication.validateAccess({ transaction })
); )
);
// If any are valid then we're done here. // If any are valid then we're done here.
if (valid) { if (valid) {
return; return;
}
} catch (err) {
// Promise.any will throw an AggregateError if all validateAccess calls throw, this would
// only be the case if all receive an unexpected error. We want to throw the first error
// as a descriptive error message.
if ("errors" in err && err.errors.length > 0) {
throw err.errors[0];
}
} }
// If all are invalid then we need to revoke the users Outline sessions. // If all are invalid then we need to revoke the users Outline sessions.