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

individual push_stream_message_ttl and push_stream_max_messages_stored_per_channel per channel #158

Open
funkydonkey123 opened this issue Sep 12, 2014 · 3 comments

Comments

@funkydonkey123
Copy link

Hi,

first of all let me thank you for this great module.
As far as I found out it is currently not possible to set push_stream_message_ttl and push_stream_max_messages_stored_per_channel for each channel individually. Or is there a way to do this?

@wandenberg
Copy link
Owner

Hi @funkydonkey123
No, these configuration are applied to all channels and stored on server main configuration.
I don't know if it makes sense to be a location configuration, or even by channel configuration.
Do you have a use case to this?

@funkydonkey123
Copy link
Author

Hi,
thanks for getting back. I have a use case where I would want to use different channels for slightly different functions (like some for live chat and some for instant notifications to visitors of a website). So at least for my use case it would be nice to be able to setup channels individually (for example: all channels where the name starts with notification the messages have a shorter ttl because the notifications are only interesting if received instantly)

@mkonecny
Copy link

mkonecny commented Jul 8, 2015

Here's one use case that would definitely justify this feature.

Currently we are using both the websocket and long-polling features.

With long-polling the client has to re-connect every 60 seconds, meaning I need to store a message in the channel in the not so rare case when the client is reconnecting.

With Websockets, since the client's connection is more permanent, it would be nice to disable message queueing.

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 a pull request may close this issue.

3 participants