Skip to content

Allow the websocket message bus to bind a tcp:// address#424

Open
mcoughlin wants to merge 2 commits into
cesium-ml:mainfrom
mcoughlin:websocket
Open

Allow the websocket message bus to bind a tcp:// address#424
mcoughlin wants to merge 2 commits into
cesium-ml:mainfrom
mcoughlin:websocket

Conversation

@mcoughlin
Copy link
Copy Markdown
Collaborator

This PR allows the websocket message bus to bind a tcp:// address (next step in Kubernetes readiness).

@mcoughlin mcoughlin requested a review from stefanv June 4, 2026 22:39
@stefanv
Copy link
Copy Markdown
Contributor

stefanv commented Jun 5, 2026

Wondering why we don't just standardize on using ports internally too, then we can avoid the munging.

Note we have two servers here: message bus + websocket server. The websocket server is, in fact, just a client of the message bus.

The message bus should be straightforward: runs a single instance, and exposes two ports — one to receive messages, one to publish messages.

The websocket server connects to the port where messages are published, and forwards them to the browser.

The app connects to the port where messages are received, and pushes them onto the bus.

So, let's just use TCP for everything—simpler?

ports:
  websocket: ...
  message_bus_receive: ...
  message_bus_publish: ...
hosts:
  message_bus: 127.0.0.1

Flow will push to hosts.message_bus:message_bus_receive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants