when new data is published to redis without an instance id, it fails to parse

Open ericraio opened this issue 10 years ago • 1 comments

If i were to publish to directly to redis with the same channel name that i use in socketcluster and my message contains "/" in it, it parses the data incorrectly, this makes it a little brittle.

I need to send my message similar to this format. which seems like a hack

uuid.v4() + "/" + messageData

Jun 01 '15 22:06 ericraio

@ericraio If you use the latest version of SC, the instance ID will always be set so this issue shouldn't come up - But I agree that the current solution is brittle.

It should use a consistent format for the message regardless of whether or not the instanceId is provided.

I've got a lot of stuff on my plate right now so a pull request would be welcome.

Jun 12 '15 13:06 jondubois