wandenberg / nginx-push-stream-module

A pure stream http push technology for your Nginx setup. Comet made easy and really scalable.
Other
2.22k stars 295 forks source link

Send messages to redis.[feature request] #20

Closed MechanisM closed 12 years ago

MechanisM commented 12 years ago

Will be great to have option to send messages in redis database from within nginx aswell. maybe as additional module or inside this one.

wandenberg commented 12 years ago

Sorry, I didn't understand your request. Can you explain better? Give some examples if possible explaining an use case to this feature.

Regards, Wandenberg

MechanisM commented 12 years ago

I mean save all sent messages into redis databse. to allow history of messages. Lets say your module used as site chat. and ppl talking in this chat. what if new visitor visit chat page? there's non messages will be displayed. so new connected ppl don't know what ppl talking about. and who is online. will be nice to have option to post into fast redis database aswell. and show smth like last 10 or last 50 messages for each connected to channel.

wandenberg commented 12 years ago

You can set the directive to store messages to "on" and when new users connect you get some old messages to display. You can choose to store messages for a certain amount of time or for a certain quantity, or mix both. This is what you need?

Has some other modules to store data on redis or memcached, you can mix one of them with pushstream module. I'm not sure if this integration should be a responsibility for this module.

Regards, Wandenberg

On Wed, Mar 28, 2012 at 7:42 PM, Eugene MechanisM < reply@reply.github.com

wrote:

I mean save all sent messages into redis databse. to allow history of messages. Lets say your module used as site chat. and ppl talking in this chat. what if new visitor visit chat page? there's non messages will be displayed. so new connected ppl don't know what ppl talking about. and who is online. will be nice to have option to post into fast redis database aswell.


Reply to this email directly or view it on GitHub:

https://github.com/wandenberg/nginx-push-stream-module/issues/20#issuecomment-4793196

MechanisM commented 12 years ago

ohhh okay. i see you point. but with redis these messages can be used/displayed anywhere. not only via nginx. so issue could be closed then coz it's jsut a feature your module partly support.