beenull/speedtest
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Removed Content-Type header from upload test because it triggered bugs in some browsers

Browse source
This commit is contained in:
adolfintel 2019-02-14 09:13:38 +01:00
parent 217421b1cf
commit e01f4fa40b
2 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Frontend/speedtest_worker.js
View file

@ -478,9 +478,7 @@ function ulTest(done) {
try{ try{
xhr[i].setRequestHeader("Content-Encoding", "identity"); // disable compression (some browsers may refuse it, but data is incompressible anyway) xhr[i].setRequestHeader("Content-Encoding", "identity"); // disable compression (some browsers may refuse it, but data is incompressible anyway)
}catch(e){} }catch(e){}
try{ //No Content-Type header in MPOT branch because it triggers bugs in some browsers
xhr[i].setRequestHeader("Content-Type", "application/octet-stream"); //force content-type to application/octet-stream in case the server misinterprets it
}catch(e){}
xhr[i].send(reqsmall); xhr[i].send(reqsmall);
} else { } else {
// REGULAR version, no workaround // REGULAR version, no workaround
@ -516,9 +514,7 @@ function ulTest(done) {
try{ try{
xhr[i].setRequestHeader("Content-Encoding", "identity"); // disable compression (some browsers may refuse it, but data is incompressible anyway) xhr[i].setRequestHeader("Content-Encoding", "identity"); // disable compression (some browsers may refuse it, but data is incompressible anyway)
}catch(e){} }catch(e){}
try{ //No Content-Type header in MPOT branch because it triggers bugs in some browsers
xhr[i].setRequestHeader("Content-Type", "application/octet-stream"); //force content-type to application/octet-stream in case the server misinterprets it
}catch(e){}
xhr[i].send(req); xhr[i].send(req);
} }
}.bind(this), }.bind(this),

2
Frontend/speedtest_worker.min.js vendored
View file

File diff suppressed because one or more lines are too long