Differential D12228

[keyserver] Don't create a thread with an admin when creating a new account
ClosedPublic
Actions

Authored by tomek on May 28 2024, 5:43 AM.

Tags

None

Referenced Files

	F2902503: D12228.id40747.diff
	Sat, Oct 5, 5:50 PM

	F2902494: D12228.id40740.diff
	Sat, Oct 5, 5:49 PM

	F2902380: D12228.id40688.diff
	Sat, Oct 5, 5:18 PM

	F2899031: D12228.id40688.diff
	Sat, Oct 5, 5:04 AM

	Unknown Object (File)
	Wed, Oct 2, 9:57 PM

	Unknown Object (File)
	Wed, Oct 2, 8:16 PM

	Unknown Object (File)
	Wed, Oct 2, 10:10 AM

	Unknown Object (File)
	Fri, Sep 27, 9:35 PM

Subscribers

Details

Reviewers

inka
ginsu

Commits

rCOMMd47601c4ec75: [keyserver] Don't create a thread with an admin when creating a new account

Summary

On a non-auth keyserver we don't have access to GENESIS and can't create a thread within it.

https://linear.app/comm/issue/ENG-7844/dont-create-a-thread-with-an-admin-when-creating-a-new-account

Depends on D12227

Test Plan

Check if the private thread is created when setting up an authoritative keyserver. Check if the thread isn't created on a non-auth keyserver.

Diff Detail

Repository

Branch

genesis

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.May 28 2024, 5:43 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMay 28 2024, 5:43 AM

tomek added a child revision: D12229: [keyserver] Throw an exception when trying to create a DM on a non-auth keyserver.May 28 2024, 5:53 AM

Harbormaster completed remote builds in B29266: Diff 40688.May 28 2024, 6:00 AM

tomek requested review of this revision.May 28 2024, 6:00 AM

ashoat added inline comments.May 28 2024, 12:19 PM

keyserver/src/creators/account-creator.js
196	Why do we always create a private thread? Does the user need one private thread per keyserver? Eventually this will be client-local, but for now I think we should only generate it on the authoritative keyserver.

tomek added inline comments.May 29 2024, 1:45 AM

keyserver/src/creators/account-creator.js
196	Ok, that makes sense - will make it conditional.

Don't create a private thread

tomek edited the test plan for this revision. (Show Details)May 29 2024, 6:31 AM

Harbormaster completed remote builds in B29279: Diff 40740.May 29 2024, 6:50 AM

inka accepted this revision.May 29 2024, 7:19 AM

This revision is now accepted and ready to land.May 29 2024, 7:19 AM

Closed by commit rCOMMd47601c4ec75: [keyserver] Don't create a thread with an admin when creating a new account (authored by tomek). · Explain WhyMay 29 2024, 7:37 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMMd47601c4ec75: [keyserver] Don't create a thread with an admin when creating a new account.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

creators/

account-creator.js

73 lines

Diff 40688

keyserver/src/creators/account-creator.js

Loading...