HomePhabricator
Diffusion Comm 395ff5e6f1a2

[native] Use recovery callServerEndpoint when fetching notif keys in…

Description

[native] Use recovery callServerEndpoint when fetching notif keys in resolveInvalidatedCookie

Summary:
This resolves ENG-5410.

The version of callServerEndpoint that resolveInvalidatedCookie gets passed is defined here. Instead of trying to dispatch a recovery attempt when it sees a session change that invalidates its cookie, this version of callServerEndpoint ignores the session change.

That solves the "double resolution" issue. It's also the right thing to use here... this overriden callServerEndpoint is used by the logIn action, and should be used by all other actions during the recovery attempt.

Depends on D9584

Test Plan: I used this patch to test. Before this diff, logs showed the first recovery attempt triggering a second one. After this diff, there was only one recovery attempt, and the initialNotificationsEncryptedMessage was resolved correctly.

Reviewers: marcin, atul

Reviewed By: atul

Subscribers: tomek, wyilio

Differential Revision: https://phab.comm.dev/D9585