Page MenuHomePhabricator
Differential D10338

[web][native] Download invite link blob and use it as an override
ClosedPublic
Actions

Authored by tomek on Dec 14 2023, 6:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 22, 6:20 AM
Unknown Object (File)
Wed, Mar 19, 2:02 AM
Unknown Object (File)
Tue, Mar 18, 8:36 AM
Unknown Object (File)
Tue, Mar 18, 8:36 AM
Unknown Object (File)
Tue, Mar 18, 8:36 AM
Unknown Object (File)
Tue, Mar 18, 8:36 AM
Unknown Object (File)
Tue, Mar 18, 8:36 AM
Unknown Object (File)
Sun, Mar 16, 12:06 AM
View All Files
Subscribers
ashoat

Details

Reviewers
michal
bartek
inka
rohan
Commits
rCOMMfd4b74535fd1: [web][native] Download invite link blob and use it as an override
Summary

Download a blob, parse it and use its content as a keyserver override.

Depends on D10311

Test Plan

Open a link on web for which a blob is downloaded and check if a blob is downloaded and its content is used correctly.

Diff Detail

Repository
rCOMM Comm
Branch
linki (branched from palys-swm/linki)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Dec 14 2023, 6:52 AM
Herald added a subscriber: ashoat. · View Herald TranscriptDec 14 2023, 6:52 AM
tomek edited the test plan for this revision. (Show Details)Dec 14 2023, 6:53 AM
Harbormaster completed remote builds in B25075: Diff 34640.Dec 14 2023, 7:45 AM
tomek requested review of this revision.Dec 14 2023, 7:45 AM
bartek added inline comments.Dec 15 2023, 5:41 AM
lib/shared/invite-links.js
32–34

The Content-Type header is redundant for the GET method because it doesn't have a request body.

39–40

This should be possible.

inka added a comment.Dec 15 2023, 8:37 AM

Can we use KeyserverOverride in useVerifyInviteLink?

rohan resigned from this revision.Dec 19 2023, 9:10 PM

I'll leave this to the other reviewers as it seems like they have some more context/feedback here

tomek planned changes to this revision.Dec 21 2023, 12:45 AM
tomek updated this revision to Diff 36531.Feb 1 2024, 8:04 AM
tomek marked 2 inline comments as done.

Rebase and address comments

Harbormaster completed remote builds in B26421: Diff 36531.Feb 1 2024, 8:23 AM
tomek updated this revision to Diff 36712.Feb 6 2024, 6:41 AM

Simplify the approach

tomek edited parent revisions, added: D10308: [keyserver] Modify link verification so that it works with users that aren't logged in to a keyserver; removed: D10311: [lib] Allow overriding a keyserver in a function returned from useKeyserverCall hook.Feb 6 2024, 6:42 AM
Harbormaster completed remote builds in B26565: Diff 36712.Feb 6 2024, 6:59 AM
bartek accepted this revision.Feb 7 2024, 1:14 AM
This revision is now accepted and ready to land.Feb 7 2024, 1:14 AM
tomek added a child revision: D10991: [native][web][lib] Extract common invite link logic.Feb 8 2024, 3:50 AM
Closed by commit rCOMMfd4b74535fd1: [web][native] Download invite link blob and use it as an override (authored by tomek). · Explain WhyFeb 14 2024, 7:58 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMfd4b74535fd1: [web][native] Download invite link blob and use it as an override.

Revision Contents
Changeset List

PathSize
lib/
shared/
41 lines
native/
navigation/
31 lines
web/
invite-links/
46 lines

Diff 34640

View Options

lib/shared/invite-links.js

Show First 20 Lines • Show All 93 Lines • ▼ Show 20 Lines
import { import {
fullStateSyncActionType, fullStateSyncActionType,
incrementalStateSyncActionType, incrementalStateSyncActionType,
} from '../types/socket-types.js'; } from '../types/socket-types.js';
import { threadPermissions } from '../types/thread-permission-types.js'; import { threadPermissions } from '../types/thread-permission-types.js';
import type { import type {
LegacyRawThreadInfo, LegacyRawThreadInfo,
MinimallyEncodedRawThreadInfos, MinimallyEncodedRawThreadInfos,
RawThreadInfos, MixedRawThreadInfos,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { import {
type ClientUpdateInfo, type ClientUpdateInfo,
processUpdatesActionType, processUpdatesActionType,
} from '../types/update-types.js'; } from '../types/update-types.js';
import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js'; import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js';
const _mapValuesWithKeys = _mapValues.convert({ cap: false }); const _mapValuesWithKeys = _mapValues.convert({ cap: false });
Show All 35 Lines
type FreshMessageStoreResult = { type FreshMessageStoreResult = {
+messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>, +messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>,
+messageStore: MessageStore, +messageStore: MessageStore,
}; };
function freshMessageStore( function freshMessageStore(
messageInfos: $ReadOnlyArray<RawMessageInfo>, messageInfos: $ReadOnlyArray<RawMessageInfo>,
truncationStatus: { [threadID: string]: MessageTruncationStatus }, truncationStatus: { [threadID: string]: MessageTruncationStatus },
currentAsOf: { +[keyserverID: string]: number }, currentAsOf: { +[keyserverID: string]: number },
threadInfos: RawThreadInfos, threadInfos: MixedRawThreadInfos,
): FreshMessageStoreResult { ): FreshMessageStoreResult {
const unshimmed = unshimMessageInfos(messageInfos); const unshimmed = unshimMessageInfos(messageInfos);
const orderedMessageInfos = sortMessageInfoList(unshimmed); const orderedMessageInfos = sortMessageInfoList(unshimmed);
const messages = _keyBy(messageID)(orderedMessageInfos); const messages = _keyBy(messageID)(orderedMessageInfos);
const messageStoreReplaceOperations = orderedMessageInfos.map( const messageStoreReplaceOperations = orderedMessageInfos.map(
messageInfo => ({ messageInfo => ({
type: 'replace', type: 'replace',
▲ Show 20 Lines • Show All 1,511 Lines • Show Last 20 Lines
View Options

native/navigation/deep-links-context-provider.react.js

Show First 20 Lines • Show All 93 Lines • ▼ Show 20 Lines
import { import {
fullStateSyncActionType, fullStateSyncActionType,
incrementalStateSyncActionType, incrementalStateSyncActionType,
} from '../types/socket-types.js'; } from '../types/socket-types.js';
import { threadPermissions } from '../types/thread-permission-types.js'; import { threadPermissions } from '../types/thread-permission-types.js';
import type { import type {
LegacyRawThreadInfo, LegacyRawThreadInfo,
MinimallyEncodedRawThreadInfos, MinimallyEncodedRawThreadInfos,
RawThreadInfos, MixedRawThreadInfos,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { import {
type ClientUpdateInfo, type ClientUpdateInfo,
processUpdatesActionType, processUpdatesActionType,
} from '../types/update-types.js'; } from '../types/update-types.js';
import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js'; import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js';
const _mapValuesWithKeys = _mapValues.convert({ cap: false }); const _mapValuesWithKeys = _mapValues.convert({ cap: false });
Show All 35 Lines
type FreshMessageStoreResult = { type FreshMessageStoreResult = {
+messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>, +messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>,
+messageStore: MessageStore, +messageStore: MessageStore,
}; };
function freshMessageStore( function freshMessageStore(
messageInfos: $ReadOnlyArray<RawMessageInfo>, messageInfos: $ReadOnlyArray<RawMessageInfo>,
truncationStatus: { [threadID: string]: MessageTruncationStatus }, truncationStatus: { [threadID: string]: MessageTruncationStatus },
currentAsOf: { +[keyserverID: string]: number }, currentAsOf: { +[keyserverID: string]: number },
threadInfos: RawThreadInfos, threadInfos: MixedRawThreadInfos,
): FreshMessageStoreResult { ): FreshMessageStoreResult {
const unshimmed = unshimMessageInfos(messageInfos); const unshimmed = unshimMessageInfos(messageInfos);
const orderedMessageInfos = sortMessageInfoList(unshimmed); const orderedMessageInfos = sortMessageInfoList(unshimmed);
const messages = _keyBy(messageID)(orderedMessageInfos); const messages = _keyBy(messageID)(orderedMessageInfos);
const messageStoreReplaceOperations = orderedMessageInfos.map( const messageStoreReplaceOperations = orderedMessageInfos.map(
messageInfo => ({ messageInfo => ({
type: 'replace', type: 'replace',
▲ Show 20 Lines • Show All 1,511 Lines • Show Last 20 Lines
View Options

web/invite-links/invite-link-handler.react.js

Show First 20 Lines • Show All 93 Lines • ▼ Show 20 Lines
import { import {
fullStateSyncActionType, fullStateSyncActionType,
incrementalStateSyncActionType, incrementalStateSyncActionType,
} from '../types/socket-types.js'; } from '../types/socket-types.js';
import { threadPermissions } from '../types/thread-permission-types.js'; import { threadPermissions } from '../types/thread-permission-types.js';
import type { import type {
LegacyRawThreadInfo, LegacyRawThreadInfo,
MinimallyEncodedRawThreadInfos, MinimallyEncodedRawThreadInfos,
RawThreadInfos, MixedRawThreadInfos,
} from '../types/thread-types.js'; } from '../types/thread-types.js';
import { import {
type ClientUpdateInfo, type ClientUpdateInfo,
processUpdatesActionType, processUpdatesActionType,
} from '../types/update-types.js'; } from '../types/update-types.js';
import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js'; import { translateClientDBThreadMessageInfos } from '../utils/message-ops-utils.js';
const _mapValuesWithKeys = _mapValues.convert({ cap: false }); const _mapValuesWithKeys = _mapValues.convert({ cap: false });
Show All 35 Lines
type FreshMessageStoreResult = { type FreshMessageStoreResult = {
+messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>, +messageStoreOperations: $ReadOnlyArray<MessageStoreOperation>,
+messageStore: MessageStore, +messageStore: MessageStore,
}; };
function freshMessageStore( function freshMessageStore(
messageInfos: $ReadOnlyArray<RawMessageInfo>, messageInfos: $ReadOnlyArray<RawMessageInfo>,
truncationStatus: { [threadID: string]: MessageTruncationStatus }, truncationStatus: { [threadID: string]: MessageTruncationStatus },
currentAsOf: { +[keyserverID: string]: number }, currentAsOf: { +[keyserverID: string]: number },
threadInfos: RawThreadInfos, threadInfos: MixedRawThreadInfos,
): FreshMessageStoreResult { ): FreshMessageStoreResult {
const unshimmed = unshimMessageInfos(messageInfos); const unshimmed = unshimMessageInfos(messageInfos);
const orderedMessageInfos = sortMessageInfoList(unshimmed); const orderedMessageInfos = sortMessageInfoList(unshimmed);
const messages = _keyBy(messageID)(orderedMessageInfos); const messages = _keyBy(messageID)(orderedMessageInfos);
const messageStoreReplaceOperations = orderedMessageInfos.map( const messageStoreReplaceOperations = orderedMessageInfos.map(
messageInfo => ({ messageInfo => ({
type: 'replace', type: 'replace',
▲ Show 20 Lines • Show All 1,511 Lines • Show Last 20 Lines