Page MenuHomePhabricator

[lib][native] Fix keyserver on master
ClosedPublic

Authored by ashoat on Jul 9 2024, 7:57 PM.
Tags
None
Referenced Files
F3566793: D12712.id42194.diff
Fri, Dec 27, 7:52 PM
F3566792: D12712.id42193.diff
Fri, Dec 27, 7:52 PM
F3566791: D12712.id42192.diff
Fri, Dec 27, 7:52 PM
F3566774: D12712.id.diff
Fri, Dec 27, 7:52 PM
F3566767: D12712.diff
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Nov 29, 4:15 PM
Unknown Object (File)
Nov 26 2024, 3:46 AM
Unknown Object (File)
Nov 23 2024, 2:00 PM
Subscribers
None

Details

Summary

D12696 and D12697 caused issues because they introduced calls to authoritativeKeyserverID() from lib/selectors/user-selectors.js, which is accessed by keyserver.

This diff moves those calls to a separate file that is only accessed by native and web. It also wraps them both in a hook for easy access.

Test Plan

Make sure keyserver runs on master again

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat published this revision for review.Jul 9 2024, 7:58 PM
ashoat added reviewers: varun, will.

Adding @will and @varun in case either of them is up right now

Going to land this without review in order to unblock the build

This revision was not accepted when it landed; it landed in state Needs Review.Jul 9 2024, 10:01 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.