fix: using a few openIds for user

namecheap · Dec 22, 2021 · 3c7cede · 3c7cede
1 parent d86f28b
commit 3c7cede
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/registry/server/auth.ts b/registry/server/auth.ts
@@ -145,10 +145,16 @@ export default (app: Express, settingsService: SettingsService, config: any): Re
                             identifiers = [identifiers];
                         }
 
-                        let user: any;
+                        let user: User | null = null;
                         for (let id of identifiers) {
-                            user = await getEntityWithCreds('openid', id, null);
-                            if (user) {
+                            const entity = await getEntityWithCreds('openid', id, null);
+                            if (!entity) {
+                                continue;
+                            }
+
+                            user = entity;
+
+                            if (entity.role === AuthRoles.admin) {
                                 break;
                             }
                         }
@@ -157,7 +163,7 @@ export default (app: Express, settingsService: SettingsService, config: any): Re
                             return done(null, false, { message: `Can\'t find presented identifiers "${identifiers.toString()}" in auth entities list` });
                         }
                         if (uidClaimName && claims[uidClaimName]) {
-                            user.identifier = claims[uidClaimName];
+                            user.identifier = claims[uidClaimName] as string;
                         }
 
                         return done(null, user);