Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[💡 FEATURE REQUEST]: Allow Skipping proxy_address in Centrifuge Plugin #2102

Open
butschster opened this issue Dec 30, 2024 · 0 comments
Open
Assignees
Labels
C-feature-request Category: feature requested, but need to be discussed

Comments

@butschster
Copy link
Contributor

Plugin

None

I have an idea!

Currently, the RoadRunner Centrifuge plugin requires both proxy_address for receiving events and grpc_api_address for sending requests. However, in a microservice architecture, only one service typically listens to events from the WebSocket (WS) server. Other services usually push messages to the WS server without needing to handle events.

Proposal:
Add the ability to skip the proxy_address configuration. If it is not set, the plugin should not create a worker pool for handling events. Instead, it should only allow sending push events through the grpc_api_address using the existing RPC interface.

This change will improve flexibility, especially in microservice setups, where only one service needs to listen to events while others focus on pushing data.

Benefits:

  • Simplifies configuration for microservice environments.
  • Reduces unnecessary resource usage on services that don’t need to listen to events.
  • Focuses each service on its specific role (e.g., event listening or pushing).

Example Configuration:

centrifuge:
  # Centrifugo server proxy address
  # Optional, only set if the service needs to listen for events
  proxy_address: null  # Skip this to disable event listening

  # gRPC server API address
  grpc_api_address: "tcp://127.0.0.1:30000"
@butschster butschster added the C-feature-request Category: feature requested, but need to be discussed label Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature-request Category: feature requested, but need to be discussed
Projects
None yet
Development

No branches or pull requests

2 participants