Issue 394 - hit statistics tool data collection (core)

lib/downloader.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 97 matching lines @@
    */
   onDownloadSuccess: null,
 
   /**
    * Callback to be triggered whenever a download fails.
    * @type Function
    */
   onDownloadError: null,
 
   /**
+   * Callback to be triggered for POST request data generation.
+   * @type Function
+   */
+  generateRequestData: null,
+
+  /**
+   * List of valid responses.
+   * @type Array
+   */
+  validResponses: [200],
+
+  /**
    * Checks whether anything needs downloading.
    */
   _doCheck: function()
   {
     let now = Date.now();
     for (let downloadable of this.dataSource())
     {
       if (downloadable.lastCheck && now - downloadable.lastCheck > this.maxAbsenceInterval)
       {
         // No checks for a long time interval - user must have been offline, e.g.
@@ skipping 115 matching lines @@
         }
 
         this.onDownloadError(downloadable, downloadUrl, error, channelStatus, responseStatus, redirectCallback);
       }
     }.bind(this);
 
     try
     {
       request = new XMLHttpRequest();
       request.mozBackgroundRequest = true;
-      request.open("GET", downloadUrl);
+      request.open(this.generateRequestData ? "POST" : "GET", downloadUrl);
     }
     catch (e)
     {
       errorCallback("synchronize_invalid_url");
       return;
     }
 
     try {
       request.overrideMimeType("text/plain");
       request.channel.loadFlags = request.channel.loadFlags |
@@ skipping 19 matching lines @@
     }.bind(this), false);
 
     request.addEventListener("load", function(event)
     {
       if (onShutdown.done)
         return;
 
       delete this._downloading[downloadable.url];
 
       // Status will be 0 for non-HTTP requests
-      if (request.status && request.status != 200)
+      if (request.status && this.validResponses.indexOf(request.status) == -1)
       {
         errorCallback("synchronize_connection_error");
         return;
       }
 
       downloadable.downloadCount++;
 
       this.onDownloadSuccess(downloadable, request.responseText, errorCallback, function redirectCallback(url)
       {
         if (redirects >= this.maxRedirects)
           errorCallback("synchronize_connection_error");
         else
         {
           downloadable.redirectURL = url;
           this._download(downloadable, redirects + 1);
         }
       }.bind(this));
     }.bind(this), false);
 
-    request.send(null);
+    if (this.generateRequestData)
+    {
+      Promise.resolve(this.generateRequestData(downloadable))
+        .then(data => {
+          request.setRequestHeader("Content-Type", "application/json");
+          request.send(JSON.stringify(data));
+        }); 
+    }
+    else
+      request.send(null);
 
     this._downloading[downloadable.url] = true;
     if (this.onDownloadStarted)
       this.onDownloadStarted(downloadable);
   },
 
   /**
    * Produces a soft and a hard expiration interval for a given supplied
    * expiration interval.
    * @return {Array} soft and hard expiration interval
@@ skipping 61 matching lines @@
    * @type Integer
    */
   hardExpiration: 0,
   
   /**
    * Number indicating how often the object was downloaded.
    * @type Integer
    */
   downloadCount: 0,
 };