[keyserver] implement function to get account prekey and prekey signature
ClosedPublic
Actions

Authored by kamil on Oct 9 2023, 6:58 AM.

Details

Reviewers

varun
marcin
ashoat

Commits

rCOMMab54e7f4f946: [keyserver] implement function to get account prekey and prekey signature

Summary

Issue: ENG-3944.

This is part of D8476 (breaking this for smaller diffs because it's complex).

Extracting code to separate function, will be used later in the stack so this reduces code duplication.

Depends on D9419

Test Plan

Flow, this only moves some code around.

Diff Detail

Repository

rCOMM Comm

Branch

publish-prekey-rotation

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Oct 9 2023, 6:58 AM

kamil held this revision as a draft.

Herald added subscribers: will, tomek. · View Herald TranscriptOct 9 2023, 6:58 AM

fix type

kamil added a child revision: D9421: [keyserver] cron job to refresh account prekeys.Oct 9 2023, 7:03 AM

Harbormaster failed remote builds in B23095: Diff 31797!Oct 9 2023, 7:12 AM

Harbormaster completed remote builds in B23096: Diff 31798.Oct 9 2023, 7:17 AM

kamil published this revision for review.Oct 9 2023, 8:18 AM

marcin accepted this revision.Oct 10 2023, 3:20 AM

This revision is now accepted and ready to land.Oct 10 2023, 3:20 AM

ashoat accepted this revision.Oct 10 2023, 7:35 AM

Closed by commit rCOMMab54e7f4f946: [keyserver] implement function to get account prekey and prekey signature (authored by kamil). · Explain WhyOct 16 2023, 2:16 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMab54e7f4f946: [keyserver] implement function to get account prekey and prekey signature.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

user/

10 lines

utils/

olm-utils.js

12 lines

Diff 31798

View Options

keyserver/src/user/login.js

	Show First 20 Lines • Show All 130 Lines • ▼ Show 20 Lines
	const styles = StyleSheet.create({			const styles = StyleSheet.create({
	dummyMessage: {			dummyMessage: {
	paddingHorizontal: 12,			paddingHorizontal: 12,
	paddingVertical: 6,			paddingVertical: 6,
	},			},
	message: {			message: {
	overflow: 'hidden',			overflow: 'hidden',
	paddingHorizontal: 12,			paddingHorizontal: 12,
	paddingVertical: 6,			paddingVertical: 8,
	},			},
	text: {			text: {
	fontFamily: 'Arial',			fontFamily: 'Arial',
	fontSize: 18,			fontSize: 14,
				benschacAuthorUnsubmitted Done Inline Actions I actually think this is a bit too small. But that's what's in the design. https://www.figma.com/file/L675ETKDnGaSwlpZAw4MIC/Mobile-App?node-id=351%3A455 @ashoat - I think 16px looks better. benschac: I actually think this is a bit too small. But that's what's in the design. https://www.figma.
				lineHeight: 22,
	},			},
	});			});

	export { InnerTextMessage, dummyNodeForTextMessageHeightMeasurement };			export { InnerTextMessage, dummyNodeForTextMessageHeightMeasurement };