wandenberg/nginx-push-stream-module — pull request #41 by karolciba (2012-07-17 16:05:34+00:00)
Title: There's no point in unescaping data that is not string

In our project we're passing around JSON objects. We've got issuses with PushStream library because onmessage callback got broken data as argument - exactly string "[object Object]" - it's happening, because parseMessage function initially converts message from JSON string (with our JSON data object inside) but then make it corrupted by unescaping it.

I've made little patch, unfortunately I wasn't been able to run test, because jshint gem is no longer available. But anyway, I didn't find any tests regarding parsing messages and I don't understand library architecture to write tests from scratch.

Best regards,
Karol Ciba