Paths

Table of Contentst

[keyserver] fix db setup
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 14, 6:47 PM

	Unknown Object (File)
	Thu, Nov 14, 2:51 PM

	Unknown Object (File)
	Mon, Nov 11, 10:33 AM

	Unknown Object (File)
	Mon, Nov 11, 10:17 AM

	Unknown Object (File)
	Mon, Nov 11, 7:27 AM

	Unknown Object (File)
	Sun, Nov 10, 11:36 PM

	Unknown Object (File)
	Thu, Nov 7, 8:01 PM

	Unknown Object (File)
	Thu, Nov 7, 5:19 PM

View All 46 Files

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.Oct 10 2024, 8:26 AM

Herald added a subscriber: tomek. · View Herald TranscriptOct 10 2024, 8:26 AM

will wait for CI before landing

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

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

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

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

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

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

Closed by commit rCOMM722342557c7f: [keyserver] fix db setup (authored by varun). · Explain WhyOct 10 2024, 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