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().
Differential D3870
[keyserver] Defer Redis publisher construction until actually used Authored by ashoat on Apr 28 2022, 3:13 AM.
Details 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(). Make sure yarn update-geoip works from a Docker buildscript
Diff Detail
|