[lib] Set default state when wiping out a keyserver store
ClosedPublic
Actions

Authored by tomek on Mar 28 2024, 3:18 PM.

Details

Reviewers

kamil
inka
varun

Commits

rCOMMb174bc063ee3: [lib] Set default state when wiping out a keyserver store

Summary

Reset the keyserver state completely.

https://linear.app/comm/issue/ENG-7070/use-default-keyserver-state-in-wipekeyserverstore

Test Plan

Added a migration that always fails by returning handleReduxMigrationFailure on web and verify that rehydrate action sets the correct keyserver store.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Mar 28 2024, 3:18 PM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 28 2024, 3:18 PM

Harbormaster completed remote builds in B27771: Diff 38460.Mar 28 2024, 3:34 PM

tomek requested review of this revision.Mar 28 2024, 3:34 PM

I think this is fine. And this also addresses the fact that we forgot to include wiping activeSessionRecovery here

This revision is now accepted and ready to land.Mar 29 2024, 8:44 AM

Closed by commit rCOMMb174bc063ee3: [lib] Set default state when wiping out a keyserver store. · Explain WhyMar 29 2024, 1:52 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMb174bc063ee3: [lib] Set default state when wiping out a keyserver store.

Revision Contents
Changeset List

Path

Size

lib/

utils/

keyserver-store-utils.js

14 lines

Diff 38540

View Options

lib/utils/keyserver-store-utils.js

[lib] Set default state when wiping out a keyserver storeClosedPublicActions