Bug 1296802 - remove LOAD_CLASSIFY_URI flag to prevent telemetry pings from running through the URL Classifier. r=francois draft
authorThomas Nguyen <tnguyen@mozilla.com>
Thu, 27 Oct 2016 18:16:26 +0800
changeset 430224 61a50f280489e5d8dd6df4e32e454369372d74a4
parent 430042 3f4c3a3cabaf94958834d3a8935adfb4a887942d
child 535157 69fe1d28e181d425f1b8fe58f56e978fb187a9d7
push id33773
push usertnguyen@mozilla.com
push dateThu, 27 Oct 2016 10:16:54 +0000
reviewersfrancois
bugs1296802
milestone52.0a1
Bug 1296802 - remove LOAD_CLASSIFY_URI flag to prevent telemetry pings from running through the URL Classifier. r=francois MozReview-Commit-ID: 3xHBa7NnhgQ
toolkit/components/telemetry/TelemetrySend.jsm
--- a/toolkit/components/telemetry/TelemetrySend.jsm
+++ b/toolkit/components/telemetry/TelemetrySend.jsm
@@ -902,16 +902,19 @@ var TelemetrySendImpl = {
 
     request.open("POST", url, true);
     request.overrideMimeType("text/plain");
     request.setRequestHeader("Content-Type", "application/json; charset=UTF-8");
     request.setRequestHeader("Date", Policy.now().toUTCString());
 
     this._pendingPingRequests.set(id, request);
 
+    // Prevent the request channel from running though URLClassifier (bug 1296802)
+    request.channel.loadFlags &= ~Ci.nsIChannel.LOAD_CLASSIFY_URI;
+
     let startTime = new Date();
     let deferred = PromiseUtils.defer();
 
     let onRequestFinished = (success, event) => {
       let onCompletion = () => {
         if (success) {
           deferred.resolve();
         } else {