-
-
Notifications
You must be signed in to change notification settings - Fork 596
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
Enable emitting to single client in managers with to=...
#1375
Comments
You can use |
I would honestly expect it the other way round 😅 , meaning you can use Also for the context, I've been building a web app that communicates with the browser-powered app: it sends notifications to the client, but can also receive certain tasks. The web app will then queue these tasks and embed a updated the PR EDIT: also updated the issue title, since it turns out emitting to a specific client is possible |
to=...
IMPORTANT: If you have a question, or you are not sure if you have found a bug in this package, then you are in the wrong place. Hit back in your web browser, and then open a GitHub Discussion instead. Likewise, if you are unable to provide the information requested below, open a discussion to troubleshoot your issue.
Describe the bug
Emitting to a
sid
from an external process (AsyncPubSubManager) will fan out the message to all the connected clients.Expected behavior
A message is only delivered to the
sid
ifto=sid
is specified.Hello and thank you for this beautiful piece of software! I really love the ergonomics and the excellent docs.
Please see PR #1374
The text was updated successfully, but these errors were encountered: