Page MenuHomePhabricator

[lib][native] Fix keyserver on master
ClosedPublic

Authored by ashoat on Jul 9 2024, 7:57 PM.
Tags
None
Referenced Files
F3370694: D12712.id42192.diff
Tue, Nov 26, 3:46 AM
Unknown Object (File)
Sat, Nov 23, 2:00 PM
Unknown Object (File)
Sat, Nov 23, 11:08 AM
Unknown Object (File)
Sun, Nov 10, 5:22 PM
Unknown Object (File)
Sun, Nov 10, 1:01 PM
Unknown Object (File)
Sun, Nov 10, 9:57 AM
Unknown Object (File)
Fri, Nov 8, 10:39 PM
Unknown Object (File)
Fri, Nov 8, 7:14 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.