[lib] render `PeerToPeerMessageHandler` in `PeerToPeerProvider`
ClosedPublic
Actions

Authored by kamil on Aug 6 2024, 5:55 AM.

Details

Reviewers

bartek
tomek

Commits

rCOMMcd384bbf13a3: [lib] render `PeerToPeerMessageHandler` in `PeerToPeerProvider`

Summary

After D12997 usePeerToPeerMessageHandler uses PeerToPeerProvider so we need to update this to be a nested component.

Depends on D12997

Test Plan

Run web & native app and make sure everything works.

Diff Detail

Repository

rCOMM Comm

Branch

resending-messages

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Aug 6 2024, 5:55 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptAug 6 2024, 5:55 AM

kamil published this revision for review.Aug 6 2024, 5:56 AM

kamil edited the summary of this revision. (Show Details)

Thanks

This revision is now accepted and ready to land.Aug 6 2024, 5:57 AM

This revision was landed with ongoing or failed builds.Aug 6 2024, 6:04 AM

Closed by commit rCOMMcd384bbf13a3: [lib] render `PeerToPeerMessageHandler` in `PeerToPeerProvider` (authored by kamil). · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMcd384bbf13a3: [lib] render `PeerToPeerMessageHandler` in `PeerToPeerProvider`.

Harbormaster completed remote builds in B30957: Diff 43179.Aug 6 2024, 6:12 AM

Revision Contents
Changeset List

Path

Size

lib/

tunnelbroker/

tunnelbroker-context.js

14 lines

Diff 43179

View Options

lib/tunnelbroker/tunnelbroker-context.js

Show First 20 Lines • Show All 573 Lines • ▼ Show 20 Lines	async keyserverAuthInfo => {

const responses: { +[string]: ClientLogInResponse } =		const responses: { +[string]: ClientLogInResponse } =
await callKeyserverEndpoint(		await callKeyserverEndpoint(
'keyserver_auth',		'keyserver_auth',
requests,		requests,
keyserverAuthCallSingleKeyserverEndpointOptions,		keyserverAuthCallSingleKeyserverEndpointOptions,
);		);

const userInfosArrays = [];

let threadInfos: RawThreadInfos = {};		let threadInfos: RawThreadInfos = {};
const calendarResult: WritableCalendarResult = {		const calendarResult: WritableCalendarResult = {
calendarQuery: keyserverAuthInfo.calendarQuery,		calendarQuery: keyserverAuthInfo.calendarQuery,
rawEntryInfos: [],		rawEntryInfos: [],
};		};
const messagesResult: WritableGenericMessagesResult = {		const messagesResult: WritableGenericMessagesResult = {
messageInfos: [],		messageInfos: [],
truncationStatus: {},		truncationStatus: {},
Show All 21 Lines	for (const keyserverID in responses) {
messagesResult.currentAsOf = {		messagesResult.currentAsOf = {
...messagesResult.currentAsOf,		...messagesResult.currentAsOf,
[keyserverID]: responses[keyserverID].serverTime,		[keyserverID]: responses[keyserverID].serverTime,
};		};
updatesCurrentAsOf = {		updatesCurrentAsOf = {
...updatesCurrentAsOf,		...updatesCurrentAsOf,
[keyserverID]: responses[keyserverID].serverTime,		[keyserverID]: responses[keyserverID].serverTime,
};		};
userInfosArrays.push(responses[keyserverID].userInfos);
userInfosArrays.push(responses[keyserverID].cookieChange.userInfos);
}		}

		const userInfosArrays = [
		responses[authoritativeKeyserverID()].userInfos,
		responses[authoritativeKeyserverID()].cookieChange.userInfos,
		tomekUnsubmitted Not Done Inline Actions I don't think it is guaranteed that there will always be a response from the authoritative keyserver, e.g. when adding a new keyserver. We probably should add a condition here and extend a test plan to verify a non-auth keyserver auth. tomek: I don't think it is guaranteed that there will always be a response from the authoritative…
		];

const userInfos = mergeUserInfos(...userInfosArrays);		const userInfos = mergeUserInfos(...userInfosArrays);

return {		return {
threadInfos,		threadInfos,
currentUserInfo: responses[authoritativeKeyserverID()]?.currentUserInfo,		currentUserInfo: responses[authoritativeKeyserverID()]?.currentUserInfo,
calendarResult,		calendarResult,
messagesResult,		messagesResult,
userInfos,		userInfos,
▲ Show 20 Lines • Show All 526 Lines • Show Last 20 Lines

[lib] render `PeerToPeerMessageHandler` in `PeerToPeerProvider`ClosedPublicActions