[keyserver] Refactoring `redisPublish` function to `messageBrokerPublish`
AbandonedPublic
Actions

Authored by tomek on Feb 10 2023, 7:31 AM.

Details

Reviewers

atul
• max

Summary

This diff introduces changes to the Keyserver codebase to refactor the redisPublish function name to the messageBrokerPublish to reflect its abstraction type. According to our task, this function will publish not only to the Redis and to the Tunnelbroker.

Linear task: ENG-2613

Test Plan

Passing the CI is enough for these changes.

Diff Detail

Repository

rCOMM Comm

Branch

ks-refactor-publish

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Feb 10 2023, 7:31 AM

• max held this revision as a draft.

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptFeb 10 2023, 7:31 AM

• max added a parent revision: D6460: [keyserver] Adding Tunnelbroker-client napi-rs publish class method.Feb 10 2023, 7:32 AM

Harbormaster completed remote builds in B16327: Diff 22329.Feb 10 2023, 7:47 AM

• max added a child revision: D6689: [keyserver] Refactoring rust-node-addon API names.Feb 10 2023, 8:02 AM

• max retitled this revision from [keyserver] Refactor `redisPublish` function to `messageBrokerPublish` to [keyserver] Refactoring `redisPublish` function to `messageBrokerPublish`.Feb 10 2023, 8:11 AM

• max edited the summary of this revision. (Show Details)Feb 13 2023, 4:47 AM

• max edited the test plan for this revision. (Show Details)

• max added reviewers: atul, tomek.

• max published this revision for review.Feb 13 2023, 5:57 AM

Seems like a straightforward rename, and looks like all usages were fixed: