Page MenuHomePhabricator

[lib][native] Fix keyserver on master
ClosedPublic

Authored by ashoat on Tue, Jul 9, 7:57 PM.
Tags
None
Referenced Files
F2320898: D12712.diff
Tue, Jul 23, 3:03 AM
F2320301: D12712.id.diff
Tue, Jul 23, 1:16 AM
Unknown Object (File)
Mon, Jul 22, 4:28 AM
Unknown Object (File)
Sat, Jul 20, 12:04 AM
Unknown Object (File)
Thu, Jul 18, 10:26 PM
Unknown Object (File)
Thu, Jul 18, 4:10 PM
Unknown Object (File)
Thu, Jul 18, 9:19 AM
Unknown Object (File)
Thu, Jul 18, 6:03 AM
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.Tue, Jul 9, 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.Tue, Jul 9, 10:01 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.