Page MenuHomePhabricator

[keyserver] Defer Redis publisher construction until actually used
ClosedPublic

Authored by ashoat on Apr 27 2022, 8:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 11, 3:33 PM
Unknown Object (File)
Thu, Nov 7, 8:35 PM
Unknown Object (File)
Thu, Nov 7, 6:50 PM
Unknown Object (File)
Wed, Nov 6, 11:41 AM
Unknown Object (File)
Wed, Oct 23, 4:28 PM
Unknown Object (File)
Wed, Oct 23, 4:28 PM
Unknown Object (File)
Tue, Oct 22, 2:05 AM
Unknown Object (File)
Mon, Oct 21, 11:28 PM

Details

Summary

The current behavior means that anything that inadvertedly imports redis.js (update-geoip.js, for instance) will crash if it can't reach Redis.

This diff makes it so RedisPublisher only tries to connect to Redis on the first call to sendMessage().

Test Plan

Make sure yarn update-geoip works from a Docker buildscript

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable