Paths

Table of Contentst

[keyserver] fix db setup
ClosedPublic
Actions

Authored by varun on Thu, Oct 10, 8:26 AM.

Tags

None

Referenced Files

	F2988059: D13693.diff
	Wed, Oct 16, 3:25 PM

	Unknown Object (File)
	Tue, Oct 15, 6:31 AM

	Unknown Object (File)
	Mon, Oct 14, 9:05 PM

	Unknown Object (File)
	Sat, Oct 12, 3:18 AM

	Unknown Object (File)
	Fri, Oct 11, 10:35 AM

	Unknown Object (File)
	Fri, Oct 11, 10:12 AM

	Unknown Object (File)
	Fri, Oct 11, 10:11 AM

	Unknown Object (File)
	Fri, Oct 11, 9:15 AM

Subscribers

tomek

Details

Reviewers

inka
kamil
ashoat

Commits

rCOMM722342557c7f: [keyserver] fix db setup

Summary

don't use fetchCallUpdateOlmAccount from verifyUserLoggedInWithoutDB

resolves https://linear.app/comm/issue/ENG-9583/running-keyserver-with-fresh-db-is-broken

Test Plan

i'm able to set up a fresh keyserver with no issues

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Thu, Oct 10, 8:26 AM

Herald added a subscriber: tomek. · View Herald TranscriptThu, Oct 10, 8:26 AM

will wait for CI before landing

varun edited the summary of this revision. (Show Details)Thu, Oct 10, 8:44 AM

Harbormaster completed remote builds in B32179: Diff 45058.Thu, Oct 10, 8:44 AM

varun added inline comments.Thu, Oct 10, 8:46 AM

keyserver/src/user/login.js
194 ↗	(On Diff #45058)	we already use the correct function, `getUpdateContentAccount`, here

ashoat accepted this revision.Thu, Oct 10, 9:50 AM

This revision is now accepted and ready to land.Thu, Oct 10, 9:50 AM

Closed by commit rCOMM722342557c7f: [keyserver] fix db setup (authored by varun). · Explain WhyThu, Oct 10, 9:51 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM722342557c7f: [keyserver] fix db setup.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

user/

2 lines

Diff 45064

View Options

keyserver/src/user/login.js

[keyserver] fix db setupClosedPublicActions