Handle user signup for OAuth and Internet Identity (#1515)

Co-Authored-By: gobdevel <99349192+gobdevel@users.noreply.github.com>

apps/api/src/app/auth/auth.module.ts

@@ -4,6 +4,7 @@ import { SubscriptionModule } from '@ghostfolio/api/app/subscription/subscriptio
 import { UserModule } from '@ghostfolio/api/app/user/user.module';
 import { ConfigurationModule } from '@ghostfolio/api/services/configuration.module';
 import { PrismaModule } from '@ghostfolio/api/services/prisma.module';
+import { PropertyModule } from '@ghostfolio/api/services/property/property.module';
 import { Module } from '@nestjs/common';
 import { JwtModule } from '@nestjs/jwt';
 
@@ -21,6 +22,7 @@ import { JwtStrategy } from './jwt.strategy';
       signOptions: { expiresIn: '180 days' }
     }),
     PrismaModule,
+    PropertyModule,
     SubscriptionModule,
     UserModule
   ],

apps/api/src/app/auth/auth.service.ts

@@ -1,5 +1,6 @@
 import { UserService } from '@ghostfolio/api/app/user/user.service';
 import { ConfigurationService } from '@ghostfolio/api/services/configuration.service';
+import { PropertyService } from '@ghostfolio/api/services/property/property.service';
 import { Injectable, InternalServerErrorException } from '@nestjs/common';
 import { JwtService } from '@nestjs/jwt';
 import { Provider } from '@prisma/client';
@@ -11,6 +12,7 @@ export class AuthService {
   public constructor(
     private readonly configurationService: ConfigurationService,
     private readonly jwtService: JwtService,
+    private readonly propertyService: PropertyService,
     private readonly userService: UserService
   ) {}
 
@@ -50,6 +52,13 @@ export class AuthService {
       });
 
       if (!user) {
+        const isUserSignupEnabled =
+          await this.propertyService.isUserSignupEnabled();
+
+        if (!isUserSignupEnabled) {
+          throw new Error('Sign up forbidden');
+        }
+
         // Create new user if not found
         user = await this.userService.createUser({
           provider,
@@ -78,6 +87,13 @@ export class AuthService {
       });
 
       if (!user) {
+        const isUserSignupEnabled =
+          await this.propertyService.isUserSignupEnabled();
+
+        if (!isUserSignupEnabled) {
+          throw new Error('Sign up forbidden');
+        }
+
         // Create new user if not found
         user = await this.userService.createUser({
           provider,

apps/api/src/app/info/info.service.ts

@@ -105,9 +105,7 @@ export class InfoService {
     }
 
     const isUserSignupEnabled =
-      ((await this.propertyService.getByKey(
+      await this.propertyService.isUserSignupEnabled();
-        PROPERTY_IS_USER_SIGNUP_ENABLED
-      )) as boolean) ?? true;
 
     if (isUserSignupEnabled) {
       globalPermissions.push(permissions.createUserAccount);

apps/api/src/app/user/user.controller.ts

@@ -70,9 +70,7 @@ export class UserController {
   @Post()
   public async signupUser(): Promise<UserItem> {
     const isUserSignupEnabled =
-      ((await this.propertyService.getByKey(
+      await this.propertyService.isUserSignupEnabled();
-        PROPERTY_IS_USER_SIGNUP_ENABLED
-      )) as boolean) ?? true;
 
     if (!isUserSignupEnabled) {
       throw new HttpException(

apps/api/src/services/property/property.service.ts

@@ -1,5 +1,8 @@
 import { PrismaService } from '@ghostfolio/api/services/prisma.service';
-import { PROPERTY_CURRENCIES } from '@ghostfolio/common/config';
+import {
+  PROPERTY_CURRENCIES,
+  PROPERTY_IS_USER_SIGNUP_ENABLED
+} from '@ghostfolio/common/config';
 import { Injectable } from '@nestjs/common';
 
 @Injectable()
@@ -39,6 +42,13 @@ export class PropertyService {
     return properties?.[aKey];
   }
 
+  public async isUserSignupEnabled() {
+    return (
+      ((await this.getByKey(PROPERTY_IS_USER_SIGNUP_ENABLED)) as boolean) ??
+      true
+    );
+  }
+
   public async put({ key, value }: { key: string; value: string }) {
     return this.prismaService.property.upsert({
       create: { key, value },