[keyserver] Defer Redis publisher construction until actually used
ClosedPublic
Actions

Authored by ashoat on Apr 27 2022, 8:13 PM.

Details

Reviewers

tomek
atul

Commits

rCOMM188caefbb4c4: [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

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/dockerize_node

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Apr 27 2022, 8:13 PM

Herald added subscribers: • abosh, • karol, • adrian. · View Herald TranscriptApr 27 2022, 8:13 PM

ashoat added a child revision: D3871: [keyserver] Dockerfile for Node keyserver.Apr 27 2022, 8:17 PM

Harbormaster completed remote builds in B8635: Diff 12025.Apr 27 2022, 8:18 PM

ashoat requested review of this revision.Apr 27 2022, 8:18 PM

tomek accepted this revision.Apr 28 2022, 1:45 AM

This revision is now accepted and ready to land.Apr 28 2022, 1:45 AM

Closed by commit rCOMM188caefbb4c4: [keyserver] Defer Redis publisher construction until actually used (authored by ashoat). · Explain WhyApr 28 2022, 11:27 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM188caefbb4c4: [keyserver] Defer Redis publisher construction until actually used.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

socket/

redis.js

11 lines

Diff 12025

View Options

keyserver/src/socket/redis.js

[keyserver] Defer Redis publisher construction until actually usedClosedPublicActions