Page MenuHomePhabricator
Differential D13693

[keyserver] fix db setup
ClosedPublic
Actions

Authored by varun on Oct 10 2024, 8:26 AM.
Tags
None
Referenced Files
F3736809: D13693.id45065.diff
Thu, Jan 9, 6:45 AM
F3736626: D13693.id45064.diff
Thu, Jan 9, 6:23 AM
F3715303: D13693.id.diff
Wed, Jan 8, 9:21 AM
Unknown Object (File)
Wed, Jan 8, 3:11 AM
Unknown Object (File)
Sun, Jan 5, 4:22 AM
Unknown Object (File)
Sat, Jan 4, 8:47 AM
Unknown Object (File)
Fri, Jan 3, 9:24 AM
Unknown Object (File)
Sat, Dec 28, 2:42 AM
View All 62 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
varun published this revision for review.Oct 10 2024, 8:30 AM
varun edited the test plan for this revision. (Show Details)

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

PathSize
keyserver/
src/
user/
2 lines

Diff 45064

View Options

keyserver/src/user/login.js

Loading...