Page MenuHomePhabricator

[lib] Fix error when deleting a keyserver
ClosedPublic

Authored by inka on Apr 23 2024, 4:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 3, 2:28 AM
Unknown Object (File)
Sun, Nov 3, 2:28 AM
Unknown Object (File)
Sun, Nov 3, 2:28 AM
Unknown Object (File)
Sun, Nov 3, 2:28 AM
Unknown Object (File)
Oct 14 2024, 8:20 PM
Unknown Object (File)
Oct 11 2024, 4:21 PM
Unknown Object (File)
Sep 12 2024, 8:54 AM
Unknown Object (File)
Sep 12 2024, 8:54 AM
Subscribers

Details

Summary

issue: ENG-7174
Deleting a keyserver would result in the app crashing, because updateConnectionStatusActionType would be dispatched for the removed keyserver.
We should just ignore this action. This happens because of unfortunate ordering - the keyserver is removed, and before the socket is unmounted the action gets dispatched

Test Plan

Tested that after this the app no longer crashes after removing a keyserver from the keyserver list.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable