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

Allow unix epoch time to be passed to push_stream_last_received_message_time #76

Closed dctrwatson closed 10 years ago

dctrwatson commented 11 years ago

This way formatting doesn't have to occur in the javascript

wandenberg commented 11 years ago

Hi @dctrwatson,

what do you mean by this? the time used on this directive is the same used on Last-Modified/If-Modified-Since headers. It was created only to support jsonp requests where you generally can't specify any additional header, it follow the same format of specified for those headers.

dctrwatson commented 11 years ago

It's for convenience for our frontend. Since I'm told we can't always mess with the headers. They have an epoch equivalent to last-modifed from a separate request, so this is just to eliminate the manual formatting (IE: https://github.com/wandenberg/nginx-push-stream-module/blob/master/misc/js/pushstream.js#L40-L43)

wandenberg commented 10 years ago

This was solved on tag 0.4.0 3d3a204177d3a7ab8a2858e04e792a6d11bf133f