Proving the user agent to octokit

Author: StanleyGoldman

octorun/src/api.js

@@ -5,7 +5,11 @@ function ApiWrapper() {
     this.octokit = octokitWrapper.createOctokit();
 
     if (!config.user || !config.token) {
-        throw "User and/or Token missing";
+        throw "user and/or token missing";
+    }
+
+    if (!config.appName) {
+        throw "appName missing";
     }
 
     this.octokit.authenticate({

octorun/src/authentication.js

@@ -9,6 +9,14 @@ var twoFactorRegex = new RegExp("must specify two-factor authentication otp code
 var badCredentialsRegex = new RegExp("bad credentials", "gi");
 
 var handleBasicAuthentication = function (username, password, onSuccess, onRequiresTwoFa, onFailure) {
+    if (!config.clientId || !config.clientSecret) {
+        throw "clientId and/or clientSecret missing";
+    }
+
+    if (!config.appName) {
+        throw "appName missing";
+    }
+
     var octokit = octokitWrapper.createOctokit();
 
     octokit.authenticate({
@@ -44,6 +52,14 @@ var handleBasicAuthentication = function (username, password, onSuccess, onRequi
 }
 
 var handleTwoFactorAuthentication = function (username, password, twoFactor, onSuccess, onFailure) {
+    if (!config.clientId || !config.clientSecret) {
+        throw "clientId and/or clientSecret missing";
+    }
+
+    if (!config.appName) {
+        throw "appName missing";
+    }
+    
     var octokit = octokitWrapper.createOctokit();
 
     octokit.authenticate({

octorun/src/bin/app-login.js

@@ -31,7 +31,13 @@ if (commander.twoFactor) {
 
         var twoFactor = readlineSync.question('Two Factor: ');
 
-        handleTwoFactorAuthentication(username, password, twoFactor);
+        try {
+            handleTwoFactorAuthentication(username, password, twoFactor);
+        }
+        catch (error) {
+            output.error(error);
+            process.exit();
+        }
     }
     else {
         var data = '';
@@ -49,7 +55,13 @@ if (commander.twoFactor) {
                 .split(/\r?\n/)
                 .filter(function (item) { return item; });
 
-            handleTwoFactorAuthentication(items[0], items[1], items[2]);
+            try {
+                handleTwoFactorAuthentication(items[0], items[1], items[2]);
+            }
+            catch (error) {
+                output.error(error);
+                process.exit();
+            }
         });
     }
 }
@@ -76,7 +88,13 @@ else {
             hideEchoBack: true
         });
 
-        handleBasicAuthentication(username, password);
+        try {
+            handleBasicAuthentication(username, password);
+        }
+        catch (error) {
+            output.error(error);
+            process.exit();
+        }
     }
     else {
         var data = '';
@@ -94,7 +112,13 @@ else {
                 .split(/\r?\n/)
                 .filter(function (item) { return item; });
 
-            handleBasicAuthentication(items[0], items[1]);
+            try {
+                handleBasicAuthentication(items[0], items[1]);
+            }
+            catch (error) {
+                output.error(error);
+                process.exit();
+            }
         });
     }
 }

octorun/src/configuration.js

@@ -2,7 +2,7 @@ require("dotenv").config({silent: true});
 
 var clientId = process.env.OCTOKIT_CLIENT_ID;
 var clientSecret = process.env.OCTOKIT_CLIENT_SECRET;
-var appName = process.env.OCTOKIT_USER_AGENT | "octorun";
+var appName = process.env.OCTOKIT_USER_AGENT;
 var user = process.env.OCTORUN_USER;
 var token = process.env.OCTORUN_TOKEN;
 

src/GitHub.Api/Application/ApplicationInfo.cs

@@ -10,7 +10,7 @@ static partial class ApplicationInfo
         public const string ApplicationName = "GitHubUnity";
         public const string ApplicationProvider = "GitHub";
 #endif
-        public const string ApplicationSafeName = "unity-internal-test";
+        public const string ApplicationSafeName = "GitHubUnity";
         public const string ApplicationDescription = "GitHub for Unity";
 
         internal static string ClientId { get; private set; } = "";

src/GitHub.Api/Tasks/OctorunTask.cs

@@ -80,7 +80,7 @@ public override void Configure(ProcessStartInfo psi)
         {
             base.Configure(psi);
 
-            psi.EnvironmentVariables.Add("OCTOKIT_USER_AGENT", ApplicationInfo.ApplicationSafeName+ AssemblyName.Version.ToString());
+            psi.EnvironmentVariables.Add("OCTOKIT_USER_AGENT", $"{ApplicationInfo.ApplicationSafeName}/{ApplicationInfo.Version}");
 
             if (clientId != null)
             {