Skip to content

Commit

Permalink
Patch our local sockjs to use a cachebusting URL for /sockjs/info.
Browse files Browse the repository at this point in the history
This has been submitted upstream too: sockjs/sockjs-client#129
  • Loading branch information
n1mmy committed Jul 25, 2013
1 parent 1ee45c4 commit b68ce66
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion packages/livedata/sockjs-0.3.4.js
Original file line number Diff line number Diff line change
Expand Up @@ -1970,7 +1970,17 @@ InfoReceiver.prototype = new EventEmitter(['finish']);
InfoReceiver.prototype.doXhr = function(base_url, AjaxObject) {
var that = this;
var t0 = (new Date()).getTime();
var xo = new AjaxObject('GET', base_url + '/info');

// <METEOR>
// https://github.com/sockjs/sockjs-client/pull/129
// var xo = new AjaxObject('GET', base_url + '/info');

var xo = new AjaxObject(
// add cachebusting parameter to url to work around a chrome bug:
// https://code.google.com/p/chromium/issues/detail?id=263981
// or misbehaving proxies.
'GET', base_url + '/info?cb=' + utils.random_string(10))
// </METEOR>

var tref = utils.delay(8000,
function(){xo.ontimeout();});
Expand Down

0 comments on commit b68ce66

Please sign in to comment.