From f94eb5b2428fcf2343b07c53f6197d21ec4787f3 Mon Sep 17 00:00:00 2001
From: Tyler Myracle <tyler.myracle@gmail.com>
Date: Sat, 13 Jan 2024 11:03:03 -0600
Subject: [PATCH] continue cleaning up and removing Auth0

---
 apps/client/pages/_app.tsx                    |  2 -
 apps/client/pages/api/auth/[...nextauth].ts   |  4 +-
 apps/client/pages/login.tsx                   |  5 +-
 .../server/src/app/routes/auth-user.router.ts |  6 --
 apps/server/src/app/routes/index.ts           |  1 -
 .../features/src/user-details/UserDetails.tsx | 15 +----
 .../src/components/dialogs/FeedbackDialog.tsx |  6 +-
 .../shared/src/providers/AuthProvider.tsx     | 64 -------------------
 .../shared/src/providers/AxiosProvider.tsx    |  6 --
 libs/client/shared/src/providers/index.ts     |  1 -
 10 files changed, 10 insertions(+), 100 deletions(-)
 delete mode 100644 apps/server/src/app/routes/auth-user.router.ts
 delete mode 100644 libs/client/shared/src/providers/AuthProvider.tsx

diff --git a/apps/client/pages/_app.tsx b/apps/client/pages/_app.tsx
index 9b608036..dc9ecf1f 100644
--- a/apps/client/pages/_app.tsx
+++ b/apps/client/pages/_app.tsx
@@ -50,8 +50,6 @@ const WithAuth = function ({ children }: PropsWithChildren) {
                     <UserAccountContextProvider>
                         <AccountContextProvider>
                             {children}
-
-                            {/* Add, edit, delete connections and manual accounts */}
                             <AccountsManager />
                         </AccountContextProvider>
                     </UserAccountContextProvider>
diff --git a/apps/client/pages/api/auth/[...nextauth].ts b/apps/client/pages/api/auth/[...nextauth].ts
index f6ecd82f..3a18412a 100644
--- a/apps/client/pages/api/auth/[...nextauth].ts
+++ b/apps/client/pages/api/auth/[...nextauth].ts
@@ -104,7 +104,9 @@ export const authOptions = {
                 const existingUser = await getAuthUserByEmail(email)
                 if (existingUser) {
                     const isPasswordMatch = await bcrypt.compare(password, existingUser.password!)
-                    if (!isPasswordMatch) throw new Error('Email or password is invalid.')
+                    if (!isPasswordMatch) {
+                        throw new Error('Email or password is invalid.')
+                    }
 
                     return existingUser
                 }
diff --git a/apps/client/pages/login.tsx b/apps/client/pages/login.tsx
index f9281725..84634495 100644
--- a/apps/client/pages/login.tsx
+++ b/apps/client/pages/login.tsx
@@ -17,7 +17,9 @@ export default function LoginPage() {
     const router = useRouter()
 
     useEffect(() => {
-        if (session) router.push('/')
+        if (session) {
+            router.push('/')
+        }
     }, [session, router])
 
     const onSubmit = async (e: React.FormEvent<HTMLFormElement>) => {
@@ -36,7 +38,6 @@ export default function LoginPage() {
         }
     }
 
-    // _app.tsx will automatically redirect if not authenticated
     return (
         <>
             <Script
diff --git a/apps/server/src/app/routes/auth-user.router.ts b/apps/server/src/app/routes/auth-user.router.ts
deleted file mode 100644
index 030e92dc..00000000
--- a/apps/server/src/app/routes/auth-user.router.ts
+++ /dev/null
@@ -1,6 +0,0 @@
-import { Router } from 'express'
-
-// Placeholder router if needed
-const router = Router()
-
-export default router
diff --git a/apps/server/src/app/routes/index.ts b/apps/server/src/app/routes/index.ts
index 5e10cf72..fd09f81c 100644
--- a/apps/server/src/app/routes/index.ts
+++ b/apps/server/src/app/routes/index.ts
@@ -2,7 +2,6 @@ export { default as accountsRouter } from './accounts.router'
 export { default as accountRollupRouter } from './account-rollup.router'
 export { default as connectionsRouter } from './connections.router'
 export { default as usersRouter } from './users.router'
-export { default as authUserRouter } from './auth-user.router'
 export { default as webhooksRouter } from './webhooks.router'
 export { default as plaidRouter } from './plaid.router'
 export { default as finicityRouter } from './finicity.router'
diff --git a/libs/client/features/src/user-details/UserDetails.tsx b/libs/client/features/src/user-details/UserDetails.tsx
index 2a0c566a..5098b7d0 100644
--- a/libs/client/features/src/user-details/UserDetails.tsx
+++ b/libs/client/features/src/user-details/UserDetails.tsx
@@ -5,12 +5,10 @@ import classNames from 'classnames'
 import { AiOutlineLoading3Quarters as LoadingIcon } from 'react-icons/ai'
 import {
     RiAnticlockwise2Line,
-    RiAppleFill,
     RiArrowGoBackFill,
     RiDownloadLine,
     RiShareForwardLine,
 } from 'react-icons/ri'
-import { UserIdentityList } from '../user-details/UserIdentityList'
 import {
     Button,
     DatePicker,
@@ -30,9 +28,8 @@ import { DeleteUserButton } from './DeleteUserButton'
 import { DateTime } from 'luxon'
 
 export function UserDetails() {
-    const { useProfile, useAuth0Profile, useUpdateProfile } = useUserApi()
+    const { useProfile, useUpdateProfile } = useUserApi()
 
-    const auth0ProfileQuery = useAuth0Profile()
     const updateProfileQuery = useUpdateProfile()
 
     const profileQuery = useProfile()
@@ -76,17 +73,7 @@ export function UserDetails() {
                                 type="text"
                             />
                         </form>
-                        {auth0ProfileQuery.data?.primaryIdentity.provider === 'apple' && (
-                            <div className="flex items-center gap-x-1 mt-2 text-gray-100">
-                                <span className="text-sm">Apple identity</span>
-                                <RiAppleFill className="w-3 h-3" />
-                            </div>
-                        )}
                     </div>
-
-                    {auth0ProfileQuery.data && (
-                        <UserIdentityList profile={auth0ProfileQuery.data} />
-                    )}
                 </LoadingPlaceholder>
             </section>
 
diff --git a/libs/client/shared/src/components/dialogs/FeedbackDialog.tsx b/libs/client/shared/src/components/dialogs/FeedbackDialog.tsx
index 4089461d..d89f90c9 100644
--- a/libs/client/shared/src/components/dialogs/FeedbackDialog.tsx
+++ b/libs/client/shared/src/components/dialogs/FeedbackDialog.tsx
@@ -1,4 +1,4 @@
-import { useAuth0 } from '@auth0/auth0-react'
+import { useSession } from 'next-auth/react'
 import { Button, Dialog } from '@maybe-finance/design-system'
 import { useState } from 'react'
 import axios from 'axios'
@@ -12,7 +12,7 @@ export interface FeedbackDialogProps {
 
 export function FeedbackDialog({ isOpen, onClose, notImplementedNotice }: FeedbackDialogProps) {
     const [feedback, setFeedback] = useState('')
-    const { user } = useAuth0()
+    const { data: session } = useSession()
 
     return (
         <Dialog isOpen={isOpen} onClose={onClose}>
@@ -42,7 +42,7 @@ export function FeedbackDialog({ isOpen, onClose, notImplementedNotice }: Feedba
                             await axios
                                 .create({ transformRequest: [(data) => JSON.stringify(data)] })
                                 .post('https://hooks.zapier.com/hooks/catch/10143005/buyo6na/', {
-                                    comment: `**From user:** ${user?.sub}\n\n${feedback}`,
+                                    comment: `**From user:** ${session?.user?.email}\n\n${feedback}`,
                                     page: `**Main app feedback**: ${window.location.href}`,
                                 })
 
diff --git a/libs/client/shared/src/providers/AuthProvider.tsx b/libs/client/shared/src/providers/AuthProvider.tsx
deleted file mode 100644
index e0254d67..00000000
--- a/libs/client/shared/src/providers/AuthProvider.tsx
+++ /dev/null
@@ -1,64 +0,0 @@
-import type { Auth0ContextInterface } from '@auth0/auth0-react'
-import { createContext } from 'react'
-import type { PropsWithChildren } from 'react'
-import { Auth0Provider, type Auth0ProviderOptions } from '@auth0/auth0-react'
-import { useRouter } from 'next/router'
-
-export const linkAuth0AccountCtx = createContext<Auth0ContextInterface | null>(
-    null
-) as Auth0ProviderOptions['context']
-
-/**
- * Auth0 Context Provider
- *
- * Why 2 configs?
- *
- * For user account linking, we need two contexts so that when the secondary
- * user is authenticated prior to linking, it doesn't log the primary user out.
- *
- * @see https://github.com/auth0/auth0-react/issues/425#issuecomment-1303619555
- */
-export function AuthProvider({ children }: PropsWithChildren) {
-    const router = useRouter()
-
-    const sharedConfig: Auth0ProviderOptions = {
-        domain: process.env.NEXT_PUBLIC_AUTH0_DOMAIN || 'REPLACE_THIS',
-        clientId: process.env.NEXT_PUBLIC_AUTH0_CLIENT_ID || 'REPLACE_THIS',
-        onRedirectCallback: (appState) => router.replace(appState?.returnTo || '/'),
-        authorizationParams: {
-            audience: process.env.NEXT_PUBLIC_AUTH0_AUDIENCE || 'https://maybe-finance-api/v1',
-            screen_hint: router.pathname === '/register' ? 'signup' : 'login',
-        },
-    }
-
-    const isBrowser = typeof window !== 'undefined'
-
-    return (
-        <Auth0Provider
-            {...sharedConfig}
-            useRefreshTokens // https://auth0.com/docs/security/tokens/refresh-tokens/configure-refresh-token-rotation
-            cacheLocation="localstorage"
-            authorizationParams={{
-                ...sharedConfig.authorizationParams,
-                redirect_uri: isBrowser ? `${window.location.origin}?primary` : undefined,
-            }}
-            skipRedirectCallback={
-                isBrowser ? window.location.href.includes('?secondary') : undefined
-            }
-        >
-            <Auth0Provider
-                {...sharedConfig}
-                authorizationParams={{
-                    ...sharedConfig.authorizationParams,
-                    redirect_uri: isBrowser ? `${window.location.origin}?secondary` : undefined,
-                }}
-                skipRedirectCallback={
-                    isBrowser ? window.location.href.includes('?primary') : undefined
-                }
-                context={linkAuth0AccountCtx}
-            >
-                {children}
-            </Auth0Provider>
-        </Auth0Provider>
-    )
-}
diff --git a/libs/client/shared/src/providers/AxiosProvider.tsx b/libs/client/shared/src/providers/AxiosProvider.tsx
index 46a709f1..38aa8d50 100644
--- a/libs/client/shared/src/providers/AxiosProvider.tsx
+++ b/libs/client/shared/src/providers/AxiosProvider.tsx
@@ -67,13 +67,7 @@ function createInstance(options?: CreateInstanceOptions) {
     return instance
 }
 
-/**
- * Injects the Auth0 access token into every axios request
- *
- * @see https://github.com/auth0/auth0-react/issues/266#issuecomment-919222402
- */
 export function AxiosProvider({ children }: PropsWithChildren) {
-    // Rather than storing access token in localStorage (insecure), we use this method to retrieve it prior to making API calls
     const API_URL = process.env.NEXT_PUBLIC_API_URL || 'http://localhost:3333'
 
     // Expose a default instance with auth, superjson, headers
diff --git a/libs/client/shared/src/providers/index.ts b/libs/client/shared/src/providers/index.ts
index 3a82ada8..89a92296 100644
--- a/libs/client/shared/src/providers/index.ts
+++ b/libs/client/shared/src/providers/index.ts
@@ -5,4 +5,3 @@ export * from './QueryProvider'
 export * from './AccountContextProvider'
 export * from './UserAccountContextProvider'
 export * from './PopoutProvider'
-export * from './AuthProvider'