19: Allow GitHub bots to use a PAT for authentication as well

Reviewed-by: ehelin
openjdk · Jun 27, 2019 · 561d715 · 561d715
1 parent a185823
commit 561d715
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/bot/src/main/java/org/openjdk/skara/bot/BotRunnerConfiguration.java b/bot/src/main/java/org/openjdk/skara/bot/BotRunnerConfiguration.java
@@ -78,10 +78,15 @@ private Map<String, Host> parseHosts(JSONObject config, Path cwd) throws Configu
                     webUriReplacement = github.get("weburl").get("replacement").asString();
                 }
 
-                var keyFile = cwd.resolve(github.get("app").get("key").asString());
-                ret.put(entry.name(), HostFactory.createGitHubHost(uri, webUriPattern, webUriReplacement, keyFile.toString(),
-                                                                   github.get("app").get("id").asString(),
-                                                                   github.get("app").get("installation").asString()));
+                if (github.contains("app")) {
+                    var keyFile = cwd.resolve(github.get("app").get("key").asString());
+                    ret.put(entry.name(), HostFactory.createGitHubHost(uri, webUriPattern, webUriReplacement, keyFile.toString(),
+                                                                       github.get("app").get("id").asString(),
+                                                                       github.get("app").get("installation").asString()));
+                } else {
+                    var pat = new PersonalAccessToken(github.get("username").asString(), github.get("pat").asString());
+                    ret.put(entry.name(), HostFactory.createGitHubHost(uri, pat));
+                }
             } else {
                 throw new ConfigurationError("Host " + entry.name());
             }