Page MenuHomePhabricator

[lib][native] Fix keyserver on master
ClosedPublic

Authored by ashoat on Jul 9 2024, 7:57 PM.
Tags
None
Referenced Files
F3048498: D12712.id42194.diff
Tue, Oct 22, 11:24 PM
F3048028: D12712.diff
Tue, Oct 22, 9:31 PM
Unknown Object (File)
Fri, Oct 11, 2:58 AM
Unknown Object (File)
Fri, Oct 11, 2:58 AM
Unknown Object (File)
Fri, Oct 11, 2:57 AM
Unknown Object (File)
Wed, Sep 25, 8:25 PM
Unknown Object (File)
Wed, Sep 25, 8:25 PM
Unknown Object (File)
Sep 18 2024, 5:33 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
Branch
ashoat/fixmaster
Lint
No Lint Coverage
Unit
No Test Coverage

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.