Merge pull request #40 from /issues/39/1

Fixes #39. Send X-Reported-With HTTP header when reporting issue.
webcompat · Sep 9, 2016 · 5418b4b · 5418b4b
2 parents 00ca97b + 6a2b39a
commit 5418b4b
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 7 deletions.
diff --git a/chrome/background.js b/chrome/background.js
@@ -25,5 +25,5 @@ function reportIssue (tab) {
   });
 }
 
-chrome.contextMenus.onClicked.addListener(function (tab) { reportIssue(tab) });
-chrome.browserAction.onClicked.addListener(function (tab) { reportIssue(tab) });
+chrome.contextMenus.onClicked.addListener(reportIssue);
+chrome.browserAction.onClicked.addListener(reportIssue);
diff --git a/firefox-webext/background.js b/firefox-webext/background.js
@@ -4,14 +4,14 @@
 
 var prefix = 'https://webcompat.com/?open=1&url=';
 var screenshotData = '';
+var reporterID = 'addon-reporter-firefox';
 
 chrome.contextMenus.create({
   id: "webcompat-contextmenu",
   title: "Report site issue",
   contexts: ["all"]
 });
 
-
 function reportIssue (tab) {
   chrome.tabs.captureVisibleTab({format: 'png'}, function(res) {
     screenshotData = res;
@@ -25,5 +25,19 @@ function reportIssue (tab) {
   });
 }
 
-chrome.contextMenus.onClicked.addListener(function (tab) { reportIssue(tab) });
-chrome.browserAction.onClicked.addListener(function (tab) { reportIssue(tab) });
+chrome.contextMenus.onClicked.addListener(reportIssue);
+chrome.browserAction.onClicked.addListener(reportIssue);
+
+// Add a custom header when the user is reporting an issue.
+chrome.webRequest.onBeforeSendHeaders.addListener(
+  function(details) {
+    details.requestHeaders.push({
+      name: 'X-Reported-With',
+      value: `${reporterID}`
+    });
+
+    return {requestHeaders: details.requestHeaders};
+  },
+  {urls: ["https://webcompat.com/?open=1&*"]},
+  ['blocking', 'requestHeaders']
+);
diff --git a/firefox-webext/manifest.json b/firefox-webext/manifest.json
@@ -23,6 +23,8 @@
     "tabs",
     "activeTab",
     "contextMenus",
+    "webRequest",
+    "webRequestBlocking",
     "https://webcompat.com/",
     "<all_urls>"
   ],

diff --git a/opera/background.js b/opera/background.js
@@ -25,5 +25,5 @@ function reportIssue (tab) {
   });
 }
 
-chrome.contextMenus.onClicked.addListener(function (tab) { reportIssue(tab) });
-chrome.browserAction.onClicked.addListener(function (tab) { reportIssue(tab) });
+chrome.contextMenus.onClicked.addListener(reportIssue);
+chrome.browserAction.onClicked.addListener(reportIssue);