diff --git a/lib/hooks/login-hooks.js b/lib/hooks/login-hooks.js --- a/lib/hooks/login-hooks.js +++ b/lib/hooks/login-hooks.js @@ -5,6 +5,7 @@ import { identityLogInActionTypes, useIdentityPasswordLogIn, + useIdentityWalletLogIn, } from '../actions/user-actions.js'; import { useKeyserverAuth } from '../keyserver-conn/keyserver-auth.js'; import { logInActionSources } from '../types/account-types.js'; @@ -27,21 +28,42 @@ const inactiveStep = { step: 'inactive' }; -function usePasswordLogIn(): ( - username: string, - password: string, -) => Promise { +type LogInInputs = + | { + +accountType: 'username', + +username: string, + +password: string, + } + | { + +accountType: 'ethereum', + +walletAddress: string, + +siweMessage: string, + +siweSignature: string, + }; + +function useLogIn(): LogInInputs => Promise { const [currentStep, setCurrentStep] = React.useState(inactiveStep); const identityPasswordLogIn = useIdentityPasswordLogIn(); + const identityWalletLogIn = useIdentityWalletLogIn(); const dispatchActionPromise = useDispatchActionPromise(); const returnedFunc = React.useCallback( - (username: string, password: string) => + (logInInputs: LogInInputs) => new Promise( // eslint-disable-next-line no-async-promise-executor async (resolve, reject) => { - const action = identityPasswordLogIn(username, password); + const action = + logInInputs.accountType === 'username' + ? identityPasswordLogIn( + logInInputs.username, + logInInputs.password, + ) + : identityWalletLogIn( + logInInputs.walletAddress, + logInInputs.siweMessage, + logInInputs.siweSignature, + ); void dispatchActionPromise(identityLogInActionTypes, action); try { await action; @@ -55,7 +77,7 @@ } }, ), - [dispatchActionPromise, identityPasswordLogIn], + [dispatchActionPromise, identityPasswordLogIn, identityWalletLogIn], ); const keyserverAuth = useKeyserverAuth(authoritativeKeyserverID()); @@ -101,4 +123,38 @@ return returnedFunc; } -export { usePasswordLogIn }; +function usePasswordLogIn(): ( + username: string, + password: string, +) => Promise { + const logIn = useLogIn(); + return React.useCallback( + (username: string, password: string) => + logIn({ + accountType: 'username', + username, + password, + }), + [logIn], + ); +} + +function useWalletLogIn(): ( + walletAddress: string, + siweMessage: string, + siweSignature: string, +) => Promise { + const logIn = useLogIn(); + return React.useCallback( + (walletAddress: string, siweMessage: string, siweSignature: string) => + logIn({ + accountType: 'ethereum', + walletAddress, + siweMessage, + siweSignature, + }), + [logIn], + ); +} + +export { usePasswordLogIn, useWalletLogIn }; diff --git a/native/account/fullscreen-siwe-panel.react.js b/native/account/fullscreen-siwe-panel.react.js --- a/native/account/fullscreen-siwe-panel.react.js +++ b/native/account/fullscreen-siwe-panel.react.js @@ -6,6 +6,7 @@ import { ActivityIndicator, View } from 'react-native'; import { setDataLoadedActionType } from 'lib/actions/client-db-store-actions.js'; +import { useWalletLogIn } from 'lib/hooks/login-hooks.js'; import { type SIWEResult, SIWEMessageTypes } from 'lib/types/siwe-types.js'; import { ServerError } from 'lib/utils/errors.js'; import { useDispatch } from 'lib/utils/redux-utils.js'; @@ -14,10 +15,7 @@ import { useGetEthereumAccountFromSIWEResult } from './registration/ethereum-utils.js'; import { RegistrationContext } from './registration/registration-context.js'; import { enableNewRegistrationMode } from './registration/registration-types.js'; -import { - useLegacySIWEServerCall, - useIdentityWalletLogInCall, -} from './siwe-hooks.js'; +import { useLegacySIWEServerCall } from './siwe-hooks.js'; import SIWEPanel from './siwe-panel.react.js'; import { commRustModule } from '../native-modules.js'; import { @@ -71,7 +69,7 @@ ); const legacySiweServerCall = useLegacySIWEServerCall(); - const identityWalletLogInCall = useIdentityWalletLogInCall(); + const identityWalletLogInCall = useWalletLogIn(); const successRef = React.useRef(false); const dispatch = useDispatch(); const onSuccess = React.useCallback( @@ -82,7 +80,11 @@ const findUserIDResponse = await commRustModule.findUserIDForWalletAddress(result.address); if (JSON.parse(findUserIDResponse).userID) { - await identityWalletLogInCall(result); + await identityWalletLogInCall( + result.address, + result.message, + result.signature, + ); } else if (enableNewRegistrationMode) { await onAccountDoesNotExist(result); } else { diff --git a/native/account/registration/existing-ethereum-account.react.js b/native/account/registration/existing-ethereum-account.react.js --- a/native/account/registration/existing-ethereum-account.react.js +++ b/native/account/registration/existing-ethereum-account.react.js @@ -6,6 +6,7 @@ import { setDataLoadedActionType } from 'lib/actions/client-db-store-actions.js'; import { siweAuthActionTypes } from 'lib/actions/siwe-actions.js'; import { useENSName } from 'lib/hooks/ens-cache.js'; +import { useWalletLogIn } from 'lib/hooks/login-hooks.js'; import { createLoadingStatusSelector } from 'lib/selectors/loading-selectors.js'; import type { SIWEResult } from 'lib/types/siwe-types.js'; import { useDispatch } from 'lib/utils/redux-utils.js'; @@ -21,10 +22,7 @@ import { useStyles } from '../../themes/colors.js'; import { UnknownErrorAlertDetails } from '../../utils/alert-messages.js'; import Alert from '../../utils/alert.js'; -import { - useIdentityWalletLogInCall, - useLegacySIWEServerCall, -} from '../siwe-hooks.js'; +import { useLegacySIWEServerCall } from '../siwe-hooks.js'; const siweAuthLoadingStatusSelector = createLoadingStatusSelector(siweAuthActionTypes); @@ -37,14 +35,18 @@ }; function ExistingEthereumAccount(props: Props): React.Node { const legacySiweServerCall = useLegacySIWEServerCall(); - const identityWalletLogInCall = useIdentityWalletLogInCall(); + const identityWalletLogInCall = useWalletLogIn(); const { params } = props.route; const dispatch = useDispatch(); const onProceedToLogIn = React.useCallback(async () => { if (usingCommServicesAccessToken) { try { - await identityWalletLogInCall(params); + await identityWalletLogInCall( + params.address, + params.message, + params.signature, + ); } catch (e) { Alert.alert( UnknownErrorAlertDetails.title, diff --git a/native/account/siwe-hooks.js b/native/account/siwe-hooks.js --- a/native/account/siwe-hooks.js +++ b/native/account/siwe-hooks.js @@ -4,8 +4,6 @@ import { siweAuth, siweAuthActionTypes } from 'lib/actions/siwe-actions.js'; import { - identityLogInActionTypes, - useIdentityWalletLogIn, identityRegisterActionTypes, useIdentityWalletRegister, } from 'lib/actions/user-actions.js'; @@ -16,10 +14,7 @@ LogInStartingPayload, LogInExtraInfo, } from 'lib/types/account-types.js'; -import type { - SIWEResult, - IdentityWalletRegisterInput, -} from 'lib/types/siwe-types.js'; +import type { IdentityWalletRegisterInput } from 'lib/types/siwe-types.js'; import { useDispatchActionPromise } from 'lib/utils/redux-promise-utils.js'; import { authoritativeKeyserverID } from '../authoritative-keyserver.js'; @@ -102,20 +97,6 @@ ); } -function useIdentityWalletLogInCall(): SIWEResult => Promise { - const identityWalletLogIn = useIdentityWalletLogIn(); - const dispatchActionPromise = useDispatchActionPromise(); - return React.useCallback( - async ({ address, message, signature }) => { - const siwePromise = identityWalletLogIn(address, message, signature); - void dispatchActionPromise(identityLogInActionTypes, siwePromise); - - await siwePromise; - }, - [dispatchActionPromise, identityWalletLogIn], - ); -} - function useIdentityWalletRegisterCall(): IdentityWalletRegisterInput => Promise { const identityWalletRegister = useIdentityWalletRegister(); const dispatchActionPromise = useDispatchActionPromise(); @@ -135,8 +116,4 @@ ); } -export { - useLegacySIWEServerCall, - useIdentityWalletLogInCall, - useIdentityWalletRegisterCall, -}; +export { useLegacySIWEServerCall, useIdentityWalletRegisterCall }; diff --git a/web/account/siwe-login-form.react.js b/web/account/siwe-login-form.react.js --- a/web/account/siwe-login-form.react.js +++ b/web/account/siwe-login-form.react.js @@ -16,12 +16,11 @@ import { identityGenerateNonceActionTypes, useIdentityGenerateNonce, - identityLogInActionTypes, - useIdentityWalletLogIn, } from 'lib/actions/user-actions.js'; import ConnectedWalletInfo from 'lib/components/connected-wallet-info.react.js'; import SWMansionIcon from 'lib/components/swmansion-icon.react.js'; import stores from 'lib/facts/stores.js'; +import { useWalletLogIn } from 'lib/hooks/login-hooks.js'; import { useLegacyAshoatKeyserverCall } from 'lib/keyserver-conn/legacy-keyserver-call.js'; import { logInExtraInfoSelector } from 'lib/selectors/account-selectors.js'; import { createLoadingStatusSelector } from 'lib/selectors/loading-selectors.js'; @@ -79,7 +78,7 @@ const legacySiweAuthCall = useLegacyAshoatKeyserverCall(siweAuth); const logInExtraInfo = useSelector(logInExtraInfoSelector); - const identityWalletLogIn = useIdentityWalletLogIn(); + const identityWalletLogIn = useWalletLogIn(); const [siweNonce, setSIWENonce] = React.useState(null); @@ -157,26 +156,25 @@ ); const attemptIdentityWalletLogIn = React.useCallback( - (walletAddress: string, siweMessage: string, siweSignature: string) => { - return dispatchActionPromise( - identityLogInActionTypes, - (async () => { - try { - return await identityWalletLogIn( - walletAddress, - siweMessage, - siweSignature, - ); - } catch (e) { - if (getMessageForException(e) === 'user not found') { - setError('account_does_not_exist'); - } - throw e; - } - })(), - ); + async ( + walletAddress: string, + siweMessage: string, + siweSignature: string, + ) => { + try { + return await identityWalletLogIn( + walletAddress, + siweMessage, + siweSignature, + ); + } catch (e) { + if (getMessageForException(e) === 'user not found') { + setError('account_does_not_exist'); + } + throw e; + } }, - [dispatchActionPromise, identityWalletLogIn], + [identityWalletLogIn], ); const dispatch = useDispatch();