Manually construct query strings for browser-request instances

Because `request` just doesn't work for us in the browser, but `browser-request` is fine despite us having to do our own query strings.

Fixes element-hq/element-web#7620

browser-index.js

@@ -1,5 +1,7 @@
 var matrixcs = require("./lib/matrix");
-matrixcs.request(require("request"));
+var request = require("browser-request");
+request.enableConstructionOfQueryString = true; // note: this is long so we hopefully don't collide
+matrixcs.request(request);
 
 // just *accessing* indexedDB throws an exception in firefox with
 // indexeddb disabled.

package.json

@@ -59,6 +59,7 @@
     "content-type": "^1.0.2",
     "loglevel": "1.6.1",
     "memfs": "^2.10.1",
+    "query-string": "^6.2.0",
     "request": "^2.88.0"
   },
   "devDependencies": {

src/http-api.js

@@ -19,6 +19,7 @@ limitations under the License.
  * @module http-api
  */
 import Promise from 'bluebird';
+const queryString = require('query-string');
 const parseContentType = require('content-type').parse;
 
 const utils = require("./utils");
@@ -746,6 +747,9 @@ module.exports.MatrixHttpApi.prototype = {
         const reqPromise = defer.promise;
 
         try {
+            if (this.opts.request.enableConstructionOfQueryString) {
+                queryParams = queryString.stringify(queryParams, opts.qsStringifyOptions);
+            }
             req = this.opts.request(
                 {
                     uri: uri,