Expose HTTPTracker to browser env.

Author: buptsb

lib/client/http-tracker.js

@@ -3,12 +3,23 @@ const bencode = require('bencode')
 const compact2string = require('compact2string')
 const debug = require('debug')('bittorrent-tracker:http-tracker')
 const get = require('simple-get')
+const querystring = require('querystring-browser')
 
 const common = require('../common')
 const Tracker = require('./tracker')
 
 const HTTP_SCRAPE_SUPPORT = /\/(announce)[^/]*$/
 
+const REQUEST_TIMEOUT = 15000
+const DESTROY_TIMEOUT = 1000
+
+const querystringStringify = obj => {
+  let ret = querystring.stringify(obj, null, null, { encodeURIComponent: escape })
+  ret = ret.replace(/[@*/+]/g, char => // `escape` doesn't encode the characters @*/+ so we do it manually
+  `%${char.charCodeAt(0).toString(16).toUpperCase()}`)
+  return ret
+}
+
 /**
  * HTTP torrent tracker client (for an individual tracker)
  *
@@ -86,7 +97,7 @@ class HTTPTracker extends Tracker {
 
     // Otherwise, wait a short time for pending requests to complete, then force
     // destroy them.
-    timeout = setTimeout(destroyCleanup, common.DESTROY_TIMEOUT)
+    timeout = setTimeout(destroyCleanup, DESTROY_TIMEOUT)
 
     // But, if all pending requests complete before the timeout fires, do cleanup
     // right away.
@@ -111,13 +122,13 @@ class HTTPTracker extends Tracker {
   _request (requestUrl, params, cb) {
     const self = this
     const u = requestUrl + (!requestUrl.includes('?') ? '?' : '&') +
-      common.querystringStringify(params)
+      querystringStringify(params)
 
     this.cleanupFns.push(cleanup)
 
     let request = get.concat({
       url: u,
-      timeout: common.REQUEST_TIMEOUT,
+      timeout: REQUEST_TIMEOUT,
       headers: {
         'user-agent': this.client._userAgent || ''
       }

package.json

@@ -48,6 +48,7 @@
   "devDependencies": {
     "@webtorrent/semantic-release-config": "1.0.5",
     "magnet-uri": "6.2.0",
+    "querystring-browser": "^1.0.4",
     "semantic-release": "17.4.3",
     "standard": "*",
     "tape": "5.2.2",

server.js

@@ -68,6 +68,10 @@ class Server extends EventEmitter {
       process.nextTick(() => {
         this.http.on('request', (req, res) => {
           if (res.headersSent) return
+          res.setHeader('Access-Control-Allow-Origin', '*')
+          res.setHeader('Access-Control-Allow-Headers', '*')
+          res.setHeader('Access-Control-Allow-Credentials', 'true')
+          res.setHeader('Access-Control-Allow-Methods', '*')
           this.onHttpRequest(req, res)
         })
       })