From 282f60e2c6454657003dea25e399696f6f88decd Mon Sep 17 00:00:00 2001 From: Tony Garnock-Jones Date: Fri, 25 May 2012 17:10:29 -0400 Subject: [PATCH] On long-haul links, the header sometimes doesn't include the padding terminator in the first chunk, so wait until it appears. --- server/web/jquery.stream-1.2.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/server/web/jquery.stream-1.2.js b/server/web/jquery.stream-1.2.js index f70017d..1bb81ef 100644 --- a/server/web/jquery.stream-1.2.js +++ b/server/web/jquery.stream-1.2.js @@ -289,7 +289,13 @@ // optional identifier within the server stream.id = text.substring(0, text.indexOf(";")); // message.index = text.indexOf(";", stream.id.length + ";".length) + ";".length; - message.index = text.indexOf(";", stream.id.length + 1) + 1; + // tonyg 20120525: If the current chunk doesn't include the end-of-padding marker, + // don't accept the header until it does. + var semiPos = text.indexOf(";", stream.id.length + 1); + if (semiPos < 0) { + return false; + } + message.index = semiPos + 1; }; handleMessage = stream.options.handleMessage || function(text, message) { // Response could contain a single message, multiple messages or a fragment of a message