[web] dispatch setAccessTokenActionType after successful identity login on web
AbandonedPublic
Actions

Authored by varun on Feb 9 2024, 8:36 PM.

Details

Reviewers

Summary

On successful identity login we should dispatch the setAccessTokenActionType action to set the CSAT in redux

Depends on D11021

Test Plan

logged in with eth wallet on web and confirmed that access token was set successfully in redux state

Repository

rCOMM Comm

Branch

siwe (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

varun requested review of this revision.Feb 9 2024, 8:52 PM

tomek requested changes to this revision.Feb 12 2024, 1:33 AM

tomek added inline comments.

web/grpc/identity-service-client-wrapper.js
293–296	It looks reasonable and convenient, but I'm worried about this approach. This function is used (through a couple of layers) in `dispatchActionPromise`, which could cause some issues: Are we allowed to call `dispatch` as a part of a promise passed to `dispatchActionPromise`? I guess that yes, but we should make sure. After the action passed to `dispatchActionPromise` is successfully resolved, e.g. login success, it is dispatched. This happens after we dispatch `setAccessTokenActionType` - is it a problem? Can this action e.g. clear a token? I think it would be safer, and probably easier to reason about, to include a token in the login success payload, but if the current approach is correct, we can keep it.

This revision now requires changes to proceed.Feb 12 2024, 1:33 AM

inka requested changes to this revision.Feb 12 2024, 2:38 AM

inka added inline comments.

web/grpc/identity-service-client-wrapper.js
293–296	login action will be clearing the access token. This is not implemented yet, because I was waiting for login actions to land, but basically we will be calling `resetUserSpecificState` on logins that are not recovery logins. This is tracked in ENG-6598 I would rather we pass the token in the login actions payload, if that's possible. Is this separate action needed? If so, we need to handle `invalidSessionRecovery` for it most likely.