From 759db40ac9f42ff71a795ceec521a7f9531d71c9 Mon Sep 17 00:00:00 2001 From: Elias Schneider Date: Tue, 11 Oct 2022 23:21:14 +0200 Subject: [PATCH] fix: wrong environment configuration for `ALLOW_REGISTRATION` --- backend/src/auth/auth.controller.ts | 2 +- frontend/src/pages/auth/signUp.tsx | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/backend/src/auth/auth.controller.ts b/backend/src/auth/auth.controller.ts index 3809ed66..e6d24b1a 100644 --- a/backend/src/auth/auth.controller.ts +++ b/backend/src/auth/auth.controller.ts @@ -21,7 +21,7 @@ export class AuthController { @Post("signUp") signUp(@Body() dto: AuthRegisterDTO) { - if (!this.config.get("ALLOW_REGISTRATION")) + if (this.config.get("ALLOW_REGISTRATION") == "false") throw new ForbiddenException("Registration is not allowed"); return this.authService.signUp(dto); } diff --git a/frontend/src/pages/auth/signUp.tsx b/frontend/src/pages/auth/signUp.tsx index 9d136e38..b4e44643 100644 --- a/frontend/src/pages/auth/signUp.tsx +++ b/frontend/src/pages/auth/signUp.tsx @@ -1,14 +1,17 @@ +import getConfig from "next/config"; import { useRouter } from "next/router"; import AuthForm from "../../components/auth/AuthForm"; import Meta from "../../components/Meta"; import useUser from "../../hooks/user.hook"; +const { publicRuntimeConfig } = getConfig(); + const SignUp = () => { const user = useUser(); const router = useRouter(); if (user) { router.replace("/"); - } else if (process.env.NEXT_PUBLIC_DISABLE_REGISTRATION) { + } else if (publicRuntimeConfig.ALLOW_REGISTRATION == "false") { router.replace("/auth/signIn"); } else { return (