Page MenuHomePhabricator

[lib][native] Fix keyserver on master
ClosedPublic

Authored by ashoat on Jul 9 2024, 7:57 PM.
Tags
None
Referenced Files
F3573609: D12712.diff
Sat, Dec 28, 2:58 PM
Unknown Object (File)
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Dec 27, 7:52 PM
Unknown Object (File)
Fri, Nov 29, 4:15 PM
Unknown Object (File)
Nov 26 2024, 3:46 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
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.