Page MenuHomePhabricator

[lib] Update genesis.id to use authoritativeKeyserverID
ClosedPublic

Authored by inka on Feb 9 2024, 7:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 1:52 AM
Unknown Object (File)
Fri, Nov 1, 10:40 PM
Unknown Object (File)
Mon, Oct 21, 1:35 AM
Unknown Object (File)
Sun, Oct 20, 11:47 PM
Unknown Object (File)
Tue, Oct 15, 6:54 PM
Unknown Object (File)
Tue, Oct 15, 6:54 PM
Unknown Object (File)
Tue, Oct 15, 6:54 PM
Unknown Object (File)
Tue, Oct 15, 6:54 PM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-6727/use-authoritativekeyserverid-in-genesis
GENESIS is the community holding DMs and thick threads for now. We want to have one GENESIS in our setups, hosted by the authoritative keyserver

Test Plan

Tested on web and native that it is possible to create a thick thread, and no errors show up.
Checked on web that the created chat has parentThreadID: "256|1", where 256 is the value of authoritativeKeyserverID in my setup.
That checks that gnesis.id is set properly, because of code in lib/actions/thread-actions.js newTread action, which if not provided a parentThreadID uses genesis.id (I made sure that was the case)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.Feb 9 2024, 8:07 AM
tomek added inline comments.
lib/facts/genesis.js
13 ↗(On Diff #36934)
This revision is now accepted and ready to land.Feb 12 2024, 1:19 AM

Turn genesis into a funtion - in lib we cannot call authoritativeKeyserverID top level, because configs are not registered yet

inka requested review of this revision.Feb 20 2024, 4:40 AM
tomek added inline comments.
keyserver/src/database/migration-config.js
605–607 ↗(On Diff #37351)

Why was the formatting changed?

This revision is now accepted and ready to land.Feb 21 2024, 1:37 AM
keyserver/src/database/migration-config.js
605–607 ↗(On Diff #37351)

This code was over 80 characters