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

There is no feedback when subscribing to a "wrong" group #10

Open
jgalazm opened this issue Jul 17, 2019 · 0 comments
Open

There is no feedback when subscribing to a "wrong" group #10

jgalazm opened this issue Jul 17, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@jgalazm
Copy link
Contributor

jgalazm commented Jul 17, 2019

For instance, when subscribing to event-Scriptqueue-stream instead of event-Scriptqueue-1-stream it throws this error:

love-manager-mount | Exception inside application: 'stream'
love-manager-mount |   File "/usr/local/lib/python3.6/site-packages/channels/consumer.py", line 59, in __call__
love-manager-mount |     [receive, self.channel_receive], self.dispatch
love-manager-mount |   File "/usr/local/lib/python3.6/site-packages/channels/utils.py", line 52, in await_many_dispatch
love-manager-mount |     await dispatch(result)
love-manager-mount |   File "/usr/local/lib/python3.6/site-packages/channels/consumer.py", line 73, in dispatch
love-manager-mount |     await handler(message)
love-manager-mount |   File "/usr/local/lib/python3.6/site-packages/channels/generic/websocket.py", line 196, in websocket_receive
love-manager-mount |     await self.receive(text_data=message["text"])
love-manager-mount |   File "/usr/local/lib/python3.6/site-packages/channels/generic/websocket.py", line 259, in receive
love-manager-mount |     await self.receive_json(await self.decode_json(text_data), **kwargs)
love-manager-mount |   File "/usr/src/love/subscription/consumers.py", line 66, in receive_json
love-manager-mount |     stream = json_data['stream']
love-manager-mount |   'stream'

This kind of exception should cause a rejection message to notify that the subscription went wrong.

@jgalazm jgalazm added the enhancement New feature or request label Jul 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant