Page MenuHomePhabricator
Differential D14033

[lib] Extract hook to send logout message to primary device
ClosedPublic
Actions

Authored by bartek on Nov 25 2024, 3:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 17, 1:15 AM
Unknown Object (File)
Wed, Apr 16, 3:48 AM
Unknown Object (File)
Wed, Apr 2, 4:43 AM
Unknown Object (File)
Thu, Mar 27, 10:49 AM
Unknown Object (File)
Thu, Mar 27, 10:24 AM
Unknown Object (File)
Wed, Mar 26, 8:57 PM
Unknown Object (File)
Tue, Mar 25, 9:33 PM
Unknown Object (File)
Mar 17 2025, 9:30 PM
View All 68 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
varun
Commits
rCOMM4e352b153d3c: [lib] Extract hook to send logout message to primary device
Summary

Part of ENG-9710.
We want to send this message not only on secondary device logout, but also on legacy logout when use_new_flow is received

Depends on D14032

Test Plan

Performed secondary device logout. Confirmed that primary device received the message.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Nov 25 2024, 3:42 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 25 2024, 3:42 AM
bartek added a child revision: D14034: [lib] Handle use_new_flow for Identity V1 logout.Nov 25 2024, 3:47 AM
Harbormaster completed remote builds in B32807: Diff 45994.Nov 25 2024, 4:14 AM
bartek published this revision for review.Nov 25 2024, 4:26 AM
kamil accepted this revision.Nov 26 2024, 1:39 AM
This revision is now accepted and ready to land.Nov 26 2024, 1:39 AM
kamil added inline comments.Nov 26 2024, 1:49 AM
lib/actions/user-actions.js
361 ↗(On Diff #45994)

I think we use this convention in the codebase

ashoat added inline comments.Nov 26 2024, 5:00 AM
lib/actions/user-actions.js
361 ↗(On Diff #45994)

I think this is okay because "logout" is a noun. On the other hand, if we were using the verb form, it would be two words: "log out"

Closed by commit rCOMM4e352b153d3c: [lib] Extract hook to send logout message to primary device (authored by bartek). · Explain WhyNov 29 2024, 5:03 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM4e352b153d3c: [lib] Extract hook to send logout message to primary device.

Revision Contents
Changeset List

PathSize
lib/
actions/
40 lines
tunnelbroker/
2 lines

Diff 46151

View Options

lib/actions/user-actions.js

// @flow // @flow
import t, { type TInterface } from 'tcomb'; import t, { type TInterface } from 'tcomb';
import type { OlmSessionInitializationInfo } from './request-types.js';
import { type AuthMetadata } from '../shared/identity-client-context.js'; import { type AuthMetadata } from '../shared/identity-client-context.js';
import { tShape } from '../utils/validation-utils.js'; import { tShape } from '../utils/validation-utils.js';
export type OLMIdentityKeys = { export type OLMIdentityKeys = {
+ed25519: string, +ed25519: string,
+curve25519: string, +curve25519: string,
}; };
const olmIdentityKeysValidator: TInterface<OLMIdentityKeys> = const olmIdentityKeysValidator: TInterface<OLMIdentityKeys> =
▲ Show 20 Lines • Show All 112 Lines • ▼ Show 20 Lines
export type OlmAPI = { export type OlmAPI = {
+initializeCryptoAccount: () => Promise<void>, +initializeCryptoAccount: () => Promise<void>,
+encrypt: (content: string, deviceID: string) => Promise<string>, +encrypt: (content: string, deviceID: string) => Promise<string>,
+decrypt: (encryptedContent: string, deviceID: string) => Promise<string>, +decrypt: (encryptedContent: string, deviceID: string) => Promise<string>,
+contentInboundSessionCreator: ( +contentInboundSessionCreator: (
contentIdentityKeys: OLMIdentityKeys, contentIdentityKeys: OLMIdentityKeys,
initialEncryptedContent: string, initialEncryptedContent: string,
) => Promise<string>, ) => Promise<string>,
+contentOutboundSessionCreator: (
contentIdentityKeys: OLMIdentityKeys,
contentInitializationInfo: OlmSessionInitializationInfo,
) => Promise<string>,
+getOneTimeKeys: (numberOfKeys: number) => Promise<OneTimeKeysResultValues>, +getOneTimeKeys: (numberOfKeys: number) => Promise<OneTimeKeysResultValues>,
+validateAndUploadPrekeys: (authMetadata: AuthMetadata) => Promise<void>, +validateAndUploadPrekeys: (authMetadata: AuthMetadata) => Promise<void>,
}; };
View Options

lib/tunnelbroker/peer-to-peer-context.js

// @flow // @flow
import t, { type TInterface } from 'tcomb'; import t, { type TInterface } from 'tcomb';
import type { OlmSessionInitializationInfo } from './request-types.js';
import { type AuthMetadata } from '../shared/identity-client-context.js'; import { type AuthMetadata } from '../shared/identity-client-context.js';
import { tShape } from '../utils/validation-utils.js'; import { tShape } from '../utils/validation-utils.js';
export type OLMIdentityKeys = { export type OLMIdentityKeys = {
+ed25519: string, +ed25519: string,
+curve25519: string, +curve25519: string,
}; };
const olmIdentityKeysValidator: TInterface<OLMIdentityKeys> = const olmIdentityKeysValidator: TInterface<OLMIdentityKeys> =
▲ Show 20 Lines • Show All 112 Lines • ▼ Show 20 Lines
export type OlmAPI = { export type OlmAPI = {
+initializeCryptoAccount: () => Promise<void>, +initializeCryptoAccount: () => Promise<void>,
+encrypt: (content: string, deviceID: string) => Promise<string>, +encrypt: (content: string, deviceID: string) => Promise<string>,
+decrypt: (encryptedContent: string, deviceID: string) => Promise<string>, +decrypt: (encryptedContent: string, deviceID: string) => Promise<string>,
+contentInboundSessionCreator: ( +contentInboundSessionCreator: (
contentIdentityKeys: OLMIdentityKeys, contentIdentityKeys: OLMIdentityKeys,
initialEncryptedContent: string, initialEncryptedContent: string,
) => Promise<string>, ) => Promise<string>,
+contentOutboundSessionCreator: (
contentIdentityKeys: OLMIdentityKeys,
contentInitializationInfo: OlmSessionInitializationInfo,
) => Promise<string>,
+getOneTimeKeys: (numberOfKeys: number) => Promise<OneTimeKeysResultValues>, +getOneTimeKeys: (numberOfKeys: number) => Promise<OneTimeKeysResultValues>,
+validateAndUploadPrekeys: (authMetadata: AuthMetadata) => Promise<void>, +validateAndUploadPrekeys: (authMetadata: AuthMetadata) => Promise<void>,
}; };