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
F2197100: D8922.diff
Fri, Jul 5, 2:15 PM
F2195067: D8922.id30248.diff
Fri, Jul 5, 9:09 AM
Unknown Object (File)
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
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
Branch
inka/fix
Lint
No Lint Coverage
Unit
No Test Coverage