Use Basic auth for GitHub (OpenUserJS#1705)

Author: joeytwiddle

libs/repoManager.js

@@ -28,7 +28,7 @@ Strategy.findOne({ name: 'github' }, function (aErr, aStrat) {
 });
 
 // Requests a GitHub url and returns the chunks as buffers
-function fetchRaw(aHost, aPath, aCallback) {
+function fetchRaw(aHost, aPath, aCallback, aOptions) {
   var options = {
     hostname: aHost,
     port: 443,
@@ -39,6 +39,14 @@ function fetchRaw(aHost, aPath, aCallback) {
     }
   };
 
+  if (aOptions) {
+    // Ideally do a deep merge of aOptions -> options
+    // But for now, we just need the headers
+    if (aOptions.headers) {
+      Object.assign(options.headers, aOptions.headers);
+    }
+  }
+
   var req = https.request(options,
     function (aRes) {
 
@@ -66,10 +74,16 @@ function fetchRaw(aHost, aPath, aCallback) {
 // Use for call the GitHub JSON api
 // Returns the JSON parsed object
 function fetchJSON(aPath, aCallback) {
-  aPath += '?client_id=' + clientId + '&client_secret=' + clientKey;
+  //aPath += '?client_id=' + clientId + '&client_secret=' + clientKey;
+  var encodedAuth = Buffer.from(`${clientId}:${client_secret}`).toString('base64');
+  var opts = {
+    headers: {
+      authorization: `Basic ${encodedAuth}`
+    }
+  };
   fetchRaw('api.github.com', aPath, function (aBufs) {
     aCallback(JSON.parse(Buffer.concat(aBufs).toString()));
-  });
+  }, opts);
 }
 
 // This manages actions on the repos of a user