Page MenuHomePhabricator

[kesyerver] Remove sessionID from website responders top level
ClosedPublic

Authored by inka on Aug 23 2023, 12:46 AM.
Tags
None
Referenced Files
F2172816: D8922.diff
Tue, Jul 2, 8:46 PM
Unknown Object (File)
Wed, Jun 19, 11:21 AM
Unknown Object (File)
Wed, Jun 19, 11:21 AM
Unknown Object (File)
Wed, Jun 19, 11:21 AM
Unknown Object (File)
Tue, Jun 18, 5:02 PM
Unknown Object (File)
Mon, Jun 17, 9:01 PM
Unknown Object (File)
Fri, Jun 14, 1:39 AM
Unknown Object (File)
Wed, Jun 12, 1:40 AM
Subscribers

Details

Summary

during https://linear.app/comm/issue/ENG-4457/refactor-sessionid-field I should have removed sessionID from top level of redux state returned from the keyserver to web client.
This field is now handled by keyserverStore.

Test Plan

Checked that the field is not present in redux state returned from the keyserver, the app still works - it's possible to have two sessions open and a message sent in one appears
in the other. Ran yarn flow-all.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable