Auth unhide (#1888)

## Describe your changes Keep the domains the same for login ## Issue ticket number and link ## Checklist before requesting a review (skip if just adding/editing APIs & templates) - [ ] I added tests, otherwise the reason is: - [ ] I added observability, otherwise the reason is: - [ ] I added analytics, otherwise the reason is:
NangoHQ · Mar 20, 2024 · c233a7e · c233a7e
1 parent 39dee4b
commit c233a7e
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 11 deletions.
diff --git a/packages/server/lib/controllers/auth.controller.ts b/packages/server/lib/controllers/auth.controller.ts
@@ -334,7 +334,7 @@ class AuthController {
             const oAuthUrl = workos?.userManagement.getAuthorizationUrl({
                 clientId: process.env['WORKOS_CLIENT_ID'] || '',
                 provider,
-                redirectUri: `${getBaseUrl()}/api/v1/login/callback`
+                redirectUri: `${getBasePublicUrl()}/api/v1/login/callback`
             });
 
             res.send({ url: oAuthUrl });
@@ -381,7 +381,7 @@ class AuthController {
             const oAuthUrl = workos?.userManagement.getAuthorizationUrl({
                 clientId: process.env['WORKOS_CLIENT_ID'] || '',
                 provider,
-                redirectUri: `${getBaseUrl()}/api/v1/login/callback`,
+                redirectUri: `${getBasePublicUrl()}/api/v1/login/callback`,
                 state: JSON.stringify(inviteParams)
             });
 

diff --git a/packages/webapp/src/pages/InviteSignup.tsx b/packages/webapp/src/pages/InviteSignup.tsx
@@ -21,8 +21,6 @@ export default function InviteSignup() {
 
     const { token } = useParams();
 
-    const tempShowAuth = false;
-
     useEffect(() => {
         const getInvite = async () => {
             const res = await getInvitee(token as string);
@@ -133,7 +131,7 @@ export default function InviteSignup() {
                                 {serverErrorMessage && <p className="mt-6 place-self-center text-sm text-red-600">{serverErrorMessage}</p>}
                             </div>
                         </form>
-                        {MANAGED_AUTH_ENABLED && tempShowAuth && (
+                        {MANAGED_AUTH_ENABLED && (
                             <>
                                 <div className="flex items-center justify-center my-4 text-xs">
                                     <div className="border-t border-gray-600 flex-grow mr-7"></div>

diff --git a/packages/webapp/src/pages/Signin.tsx b/packages/webapp/src/pages/Signin.tsx
@@ -35,8 +35,6 @@ export default function Signin() {
         }
     };
 
-    const tempShowAuth = false;
-
     return (
         <>
             <DefaultLayout>
@@ -89,7 +87,7 @@ export default function Signin() {
                                 {serverErrorMessage && <p className="mt-6 place-self-center text-sm text-red-600">{serverErrorMessage}</p>}
                             </div>
 
-                            {MANAGED_AUTH_ENABLED && tempShowAuth && (
+                            {MANAGED_AUTH_ENABLED && (
                                 <>
                                     <div className="flex items-center justify-center my-4 text-xs">
                                         <div className="border-t border-gray-600 flex-grow mr-7"></div>

diff --git a/packages/webapp/src/pages/Signup.tsx b/packages/webapp/src/pages/Signup.tsx
@@ -45,8 +45,6 @@ export default function Signup() {
         }
     };
 
-    const tempShowAuth = false;
-
     return (
         <>
             <DefaultLayout>
@@ -110,7 +108,7 @@ export default function Signup() {
                                 {serverErrorMessage && <p className="mt-6 place-self-center text-sm text-red-600">{serverErrorMessage}</p>}
                             </div>
                         </form>
-                        {MANAGED_AUTH_ENABLED && tempShowAuth && (
+                        {MANAGED_AUTH_ENABLED && (
                             <>
                                 <div className="flex items-center justify-center my-4 text-xs">
                                     <div className="border-t border-gray-600 flex-grow mr-7"></div>