Page MenuHomePhabricator

[Keyserver] Use getCommConfig for tunnelbroker connection
ClosedPublic

Authored by kamil on Aug 25 2023, 1:00 PM.
Tags
None
Referenced Files
F3646376: D8957.id31046.diff
Sat, Jan 4, 9:33 PM
F3646375: D8957.id31045.diff
Sat, Jan 4, 9:33 PM
F3646373: D8957.id30337.diff
Sat, Jan 4, 9:33 PM
F3640259: D8957.id30332.diff
Sat, Jan 4, 11:34 AM
F3640258: D8957.id.diff
Sat, Jan 4, 11:34 AM
Unknown Object (File)
Fri, Jan 3, 2:05 PM
Unknown Object (File)
Thu, Jan 2, 8:26 PM
Unknown Object (File)
Thu, Jan 2, 4:47 PM
Subscribers

Details

Summary

Allow for the tunnelbroker endpoint to be configured. Necessary
for targeting production or staging instance during deployments.

https://linear.app/comm/issue/ENG-4706

Depends on D8956

Test Plan
cd keyserver
yarn dev # Should say defaulting to local instance
export COMM_JSONCONFIG_facts_tunnelbroker="{\"url\":\"ws://127.0.0.1:51001\"}"
yarn dev # Should omit default message, still connect to local instance

Diff Detail

Repository
rCOMM Comm
Branch
tb-config
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 25 2023, 1:13 PM
Harbormaster failed remote builds in B22098: Diff 30332!
This revision is now accepted and ready to land.Aug 28 2023, 8:25 AM
This revision now requires review to proceed.Aug 30 2023, 1:01 PM
ashoat added inline comments.
keyserver/src/socket/tunnelbroker.js
44 ↗(On Diff #30337)

Can this be done in parallel instead of waiting for the await on line 42 to complete

This revision is now accepted and ready to land.Sep 6 2023, 6:13 PM
ashoat edited reviewers, added: jon; removed: ashoat.
ashoat foisted this revision upon kamil.
ashoat edited reviewers, added: ashoat; removed: kamil.

@kamil, would you mind taking over this diff, addressing my inline comment, and landing it?

@kamil, would you mind taking over this diff, addressing my inline comment, and landing it?

Sure, I'll address the comment before landing

  • address review
  • review before landing