open.source/manifold
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove unnecessary new user creation promise machinery

Browse Source
This commit is contained in:
Marshall Polaris 2022-07-20 18:18:53 -07:00
parent b130e0e642
commit 344946f84f
1 changed files with 5 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
web/components/auth-context.tsx
View File

@ -16,9 +16,6 @@ import { useStateCheckEquality } from 'web/hooks/use-state-check-equality'
const CACHED_USER_KEY = 'CACHED_USER_KEY'
// used to avoid weird race condition
let createUserPromise: Promise<User> | undefined = undefined
export const AuthContext = createContext<User | null>(null)
export function AuthProvider({ children }: any) {
@ -32,17 +29,12 @@ export function AuthProvider({ children }: any) {
if (fbUser) {
let user: User | null = await getUser(fbUser.uid)
if (!user) {
if (createUserPromise == null) {
let deviceToken = localStorage.getItem('device-token')
if (!deviceToken) {
deviceToken = randomString()
localStorage.setItem('device-token', deviceToken)
}
createUserPromise = createUser({ deviceToken }).then(
(r) => r as User
)
let deviceToken = localStorage.getItem('device-token')
if (!deviceToken) {
deviceToken = randomString()
localStorage.setItem('device-token', deviceToken)
}
user = await createUserPromise
user = await createUser({ deviceToken }).then((r) => r as User)
}
setCurrentUser(user)
// Persist to local storage, to reduce login blink next time.
@ -53,7 +45,6 @@ export function AuthProvider({ children }: any) {
} else {
// User logged out; reset to null
setCurrentUser(null)
createUserPromise = undefined
localStorage.removeItem(CACHED_USER_KEY)
deleteAuthCookies()
}