HomePhabricator
Diffusion Comm 188caefbb4c4

[keyserver] Defer Redis publisher construction until actually used

Description

[keyserver] Defer Redis publisher construction until actually used

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

Reviewers: palys-swm, atul

Reviewed By: palys-swm

Subscribers: Adrian, karol-bisztyga, yayabosh

Differential Revision: https://phabricator.ashoat.com/D3870

Details

Provenance
ashoatAuthored on Apr 27 2022, 8:09 PM
Reviewer
tomek
Differential Revision
D3870: [keyserver] Defer Redis publisher construction until actually used
Parents
rCOMMbdc1a21f0f05: [keyserver] Remove links from docker-compose.yml
Branches
Unknown
Tags
Unknown