Skip to content

Commit 5dc7b6f

Browse files
committed
Update triggers.js
1 parent cba81dd commit 5dc7b6f

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

src/triggers.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -736,16 +736,10 @@ async function builtInTriggerValidator(options, request, auth) {
736736
const userRoles = options.requireUserRoles;
737737
if (userRoles) {
738738
const roles = await auth.getUserRoles();
739-
const validateRoles = () => {
740-
const roleNames = roles.map(role => role.replace('role:', ''));
741-
for (const role of userRoles) {
742-
if (roleNames.includes(role)) {
743-
return;
744-
}
745-
throw `Validation failed. User does not match the required roles.`;
746-
}
747-
};
748-
validateRoles();
739+
const hasRole = userRoles.some(requiredRole => roles.includes(`role:${requiredRole}`));
740+
if (!hasRole) {
741+
throw `Validation failed. User does not match the required roles.`;
742+
}
749743
}
750744
const userKeys = options.requireUserKeys || [];
751745
if (Array.isArray(userKeys)) {

0 commit comments

Comments
 (0)