Paths

Table of Contentst

Differential D10286

[keyserver] Update `rawThreadInfoFromServerThreadInfo` to support `minimallyEncodePermissions` option
ClosedPublic
Actions

Authored by atul on Dec 10 2023, 7:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 25, 1:44 PM

	Unknown Object (File)
	Mon, Nov 18, 11:09 AM

	Unknown Object (File)
	Wed, Nov 13, 4:37 AM

	Unknown Object (File)
	Wed, Nov 6, 1:54 PM

	Unknown Object (File)
	Oct 20 2024, 6:33 AM

	Unknown Object (File)
	Oct 15 2024, 10:25 PM

	Unknown Object (File)
	Oct 15 2024, 10:25 PM

	Unknown Object (File)
	Oct 15 2024, 10:25 PM

View All 74 Files

Subscribers

None

Details

Reviewers

ashoat
ginsu
tomek
rohan

Commits

rCOMMab7a55637495: [keyserver] Update `rawThreadInfoFromServerThreadInfo` to support…

Summary

This is the diff that finally allows keyserver to send the new "minimally encoded" permissions to new clients.

We introduce minimallyEncodePermissions option, which for now will be set to NATIVE FUTURE_CODE_VERSION. Once we have a version of client that fully supports minimally encoded permissions, we will update this setting.

Depends on D10285

Test Plan

Will set breakpoint in keyserver and ensure that rawThreadInfos are constructed as expected.

642b40.png (2×2 px, 531 KB)

Will check Redux dev tools on corresponding client to ensure that minimally encoded permissions appear as expected.
Will play around with the native app and make sure that permissions checks continue to behave as expected.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 10 2023, 7:22 PM

atul published this revision for review.Dec 10 2023, 7:22 PM

Harbormaster completed remote builds in B24932: Diff 34466.Dec 10 2023, 7:41 PM

atul edited the test plan for this revision. (Show Details)Dec 10 2023, 7:55 PM

rohan accepted this revision.Dec 11 2023, 7:26 AM

This revision is now accepted and ready to land.Dec 11 2023, 7:26 AM

fix

atul added a child revision: D10294: [lib][keyserver] Replace `legacyRawThreadInfoValidator` with `rawThreadInfoValidator`.Dec 11 2023, 11:58 AM

Harbormaster completed remote builds in B24950: Diff 34487.Dec 11 2023, 12:01 PM

last rebase pass

Harbormaster completed remote builds in B25053: Diff 34613.Dec 13 2023, 4:05 PM

Closed by commit rCOMMab7a55637495: [keyserver] Update `rawThreadInfoFromServerThreadInfo` to support… (authored by atul). · Explain WhyDec 14 2023, 1:03 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMab7a55637495: [keyserver] Update `rawThreadInfoFromServerThreadInfo` to support….

Revision Contents
Changeset List

Path

Size

keyserver/

src/

fetchers/

thread-fetchers.js

18 lines

responders/

user-responders.js

3 lines

lib/

shared/

thread-utils.js

11 lines

Diff 34678

keyserver/src/fetchers/thread-fetchers.js

Loading...

keyserver/src/responders/user-responders.js

Loading...

lib/shared/thread-utils.js

Loading...