Page MenuHomePhabricator

[lib] Make possible to mark ks as temporarily connected
AbandonedPublic

Authored by inka on Dec 29 2023, 5:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 3:34 PM
Unknown Object (File)
Tue, Nov 26, 11:59 AM
Unknown Object (File)
Thu, Oct 31, 12:51 AM
Unknown Object (File)
Oct 16 2024, 2:10 AM
Unknown Object (File)
Oct 16 2024, 2:10 AM
Unknown Object (File)
Oct 16 2024, 2:09 AM
Unknown Object (File)
Oct 16 2024, 2:09 AM
Unknown Object (File)
Sep 23 2024, 10:39 AM
Subscribers

Details

Reviewers
ginsu
michal
kamil
Summary

issue: https://linear.app/comm/issue/ENG-6300/add-connectionissue-marking-the-keyserver-as-temporary
We want to be able to mark a keyserver as "temporarily connected"

If VERSION or VERIFY_INVITE_LINK endpoints were called, we should somehow mark the keyserver as “temporarily” connected - keyserverConnectionHandler will not be attempting to log the user in to that keyserver, and this keyserver will not be persisted.

  1. It might be useful to use the same cookie that we got from the keyserver, in the next request
  2. But calling those endpoints doesn’t necessarily mean that the user wants to permanently connect (login) to this keyserver. The user might for example resign from accepting an invite link.
Test Plan

Tested that setting temporary: true results in connectionIssue for the given keyserver being set to 'temporarily_connected'

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage