search

wandenberg / nginx-push-stream-module

A pure stream http push technology for your Nginx setup. Comet made easy and really scalable.
Other
2.22k stars 295 forks source link

Update pushstream.js for correct work with IE9 #120

Closed apsavin closed 9 years ago

apsavin commented 10 years ago

IE9 causes error when you trying to read xhr props after xhr.abort(). So, we must remove onreadystatechange of xhr to prevent this errors. Also, LongPollingWrapper#_internalListen method was fixed: we must send new response after abort of previous xhr.

wandenberg commented 10 years ago

Hi @apsavin

how can I reproduce the error? I will try to put this scenario on the js tests.

apsavin commented 10 years ago
  1. Open page with long-polling connect in IE 9 (or IE 10 in IE 9 mode)
  2. Wait while pushstream run xhr.abort() at line 247 of pushstream.js
  3. xhr.onreadystatechange will run; every ready attempt of xhr properties, except readyState, will cause c00c023f error