[jQuery] streaming, opera breaks on junk split + FIX

[PavelR577]

when junk is getting broadcasted in parts opera will fail to establish connection.

Here is the fixed code:

in function _executeRequest(request)

if (!_response.junkFull && (text.indexOf("<!-- Welcome to the Atmosphere Framework.") == -1 || text.indexOf("<!-- EOD -->") == -1)) {
    if (!jQuery.browser.opera)
        return;
} else
    _response.junkFull = true;

and in the bottom:

 if (jQuery.browser.opera) {
    jQuery.atmosphere.iterate(function() {
        if (ajaxRequest.responseText.length > rq.lastIndex) {
            try {
                _response.status = ajaxRequest.status;
                _response.headers = parseHeaders(ajaxRequest.getAllResponseHeaders());

                // HOTFIX for firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=608735
                if (_request.readResponsesHeaders && _request.headers) {
                    jQuery.each(_request.headers, function(name) {
                        var v = ajaxRequest.getResponseHeader(name);
                        if (v) {
                            _response.headers[name] = v;
                        }
                    });
                }
            }
            catch(e) {
                _response.status = 404;
            }
//todo: fix for junk comming in parts in opera
            if (!_response.junkFull) {
                var endOfJunk = "<!-- EOD -->";
                var endOfJunkLength = endOfJunk.length;
                var junkEnd = ajaxRequest.responseText.indexOf(endOfJunk) + endOfJunkLength;
                rq.lastIndex = junkEnd; //skip to end of junk
                _response.junkFull = true;
            } else {
                //any message from the server will reset the last ping time
                rq.lastPingTime = (new Date()).getTime();
                _response.state = "messageReceived";
                _response.responseBody = ajaxRequest.responseText.substring(rq.lastIndex);
                rq.lastIndex = ajaxRequest.responseText.length;

                _invokeCallback();
                if ((rq.transport == 'streaming') && (ajaxRequest.responseText.length > rq.maxStreamingLength)) {
                    // Close and reopen connection on large data received
                    ajaxRequest.abort();
                    _doRequest(ajaxRequest, rq, true);
                }
            }
        }
    }, 0);
}

[jfarcand]

Salut, it is hard to find what you really changed. I strongly recommend you do a pull request if you can, or at least attach a git diff of the atmosphere.js file. Thanks!!!

[jfarcand]

Patch applied, thanks!