[atmosphere.js] JSONP not working

[jfarcand]

The new atmosphere.js script (the one without jquery) fail on the second request when using JSONP as transport. The same error occurs with portal.js.

[flowersinthesand]

When executing portal's test suite, it turns out longpolljsonp that is a transport name in the portal of JSONP works correctly. With the chat sample from atmosphere-samples and 'jsonp' transport, first jsonp request's response is as follows.

atmosphere1378902554401({"message" : "53|{"message":"fef","author":"fef","time":1378902556941}"});

As you can see the above code's color, this is malformed JavaScript code.

After modifying this correctly and executing it in the console, I was able to see that next request fires and its response is wrongly formatted.

[jfarcand]

Ah! OK let me fix that.

[jfarcand]

OK I think I'm missing something. Event if I force Jackson to encore the time in String (that shoudn't make any difference

atmosphere1378904720644({"message" : "59|{"message":"dsada","author":"dsada","time":"1378904722083"}"});

I still see an error. Is it because of the '|' . Doing the same with jQuery seems to work so I'm a little puzzled.

[flowersinthesand]

This is the expected result, and of course the time property is not problem.

atmosphere1378902554401({"message" : "53|{\"message\":\"fef\",\"author\":\"fef\",\"time\":1378902556941}"});

With the latest jquery.atmosphere.js, I see the same error.

jQuery190069097618936723_1378906773365({"message" : "53|{"message":"aaa","author":"aaa","time":1378906779316}"});

[jfarcand]

OK closing the issue, will re-open another one as it work with the jersey-pubsub sample.