Merge pull request #287 from jglick/appID

Added doCheckAppID
jenkinsci · Mar 24, 2020 · 33d67ae · 33d67ae
2 parents 75d5dc6 + dfb37ac
commit 33d67ae
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubAppCredentials.java b/src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubAppCredentials.java
@@ -157,6 +157,17 @@ public ListBoxModel doFillApiUriItems() {
             return getPossibleApiUriItems();
         }
 
+        public FormValidation doCheckAppID(@QueryParameter String appID) {
+            if (!appID.isEmpty()) {
+                try {
+                    Integer.parseInt(appID);
+                } catch (NumberFormatException x) {
+                    return FormValidation.warning("An app ID is likely to be a number, distinct from the app name");
+                }
+            }
+            return FormValidation.ok();
+        }
+
         @POST
         @SuppressWarnings("unused") // stapler
         @Restricted(NoExternalUse.class) // stapler