Better notice that the JSON object is missing

[statico]

I was testing an app on IE6 and my SocketIO listener was receiving messages like {'$error': 'something}. It turns out that this is because IE6 has no JSON object built-in and SocketIO's stringify() function. Adding json2.js solved the problem.

It's be nice if that error message said something like, "Sorry, no JSON object available, can't stringify."

[rauchg]

Ok. Have had a few reports like this. Will make it more clear. Thanks!

[statico]

Awesome!

[3rd-Eden]

What we could do is something like:

('console' in window && console.error ? console.error : alert )('Trying to encode as JSON, but JSON.stringify is missing.');

Which might be bit dirty but as developer you will notice it when you test your application

[statico]

If Socket.IO won't work at all without JSON present, then this makes sense.

I'd add "Socket.IO Error:" at the beginning of the string. If an IE6 user complains to the app's developer, the developer will have a hint as to what's causing the error. :)

[3rd-Eden]

Socket.io only requires JSON support if you are actually sending that needs to be transformed in to a string before send it.

Maybe in a future release we could add better error handling and create a dedicated .on('error', function(error){}) listener for issues like these.

[3rd-Eden]

Added to pull request 104: socketio/socket.io#104