Fixed issue with first user not being made admin

EtherealEngine · Oct 23, 2023 · b137218 · b137218
1 parent d92b107
commit b137218
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/server-core/src/user/identity-provider/identity-provider.hooks.ts b/packages/server-core/src/user/identity-provider/identity-provider.hooks.ts
@@ -130,7 +130,7 @@ async function addIdentityProviderType(context: HookContext<IdentityProviderServ
     context.params!.provider &&
     !['password', 'email', 'sms'].includes((context!.data as IdentityProviderData).type)
   ) {
-    ;(context.data as IdentityProviderData).type = 'guest'
+    ;(context.data as IdentityProviderData).type = 'guest' //Non-password/magiclink create requests must always be for guests
   }
 
   const adminScopes = await context.app.service(scopePath).find({
@@ -139,7 +139,7 @@ async function addIdentityProviderType(context: HookContext<IdentityProviderServ
     }
   })
 
-  if (adminScopes.total === 0 && isDev && (context.data as IdentityProviderData).type !== 'guest') {
+  if (adminScopes.total === 0 && (isDev || (context.data as IdentityProviderData).type !== 'guest')) {
     ;(context.data as IdentityProviderData).type = 'admin'
   }
 }