Merge pull request #576 from Vlatombe/historicalbuild

Whitelist method moved from `Run#getFullDisplayName` to `HistoricalBuild#getFullDisplayName`
jenkinsci · Sep 11, 2024 · 9131007 · 9131007
2 parents b26663c + b12683a
commit 9131007
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/main/resources/org/jenkinsci/plugins/scriptsecurity/sandbox/whitelists/jenkins-whitelist b/src/main/resources/org/jenkinsci/plugins/scriptsecurity/sandbox/whitelists/jenkins-whitelist
@@ -36,4 +36,5 @@ method hudson.tools.ToolInstallation getHome
 method hudson.tools.ToolInstallation getName
 method jenkins.model.CauseOfInterruption getShortDescription
 method jenkins.model.CauseOfInterruption$UserInterruption getUserId
+method jenkins.model.HistoricalBuild getFullDisplayName
 staticField jenkins.model.Jenkins VERSION
diff --git a/...est/java/org/jenkinsci/plugins/scriptsecurity/sandbox/whitelists/StaticWhitelistTest.java b/...est/java/org/jenkinsci/plugins/scriptsecurity/sandbox/whitelists/StaticWhitelistTest.java
@@ -155,7 +155,9 @@ static void sanity(URL definition) throws Exception {
             new MethodSignature(MatchResult.class, "group", String.class),
             new MethodSignature(MatchResult.class, "hasMatch"),
             new MethodSignature(MatchResult.class, "namedGroups"),
-            new MethodSignature(MatchResult.class, "start", String.class)
+            new MethodSignature(MatchResult.class, "start", String.class),
+            // TODO Do not exist until Jenkins includes https://github.com/jenkinsci/jenkins/pull/9674
+            new MethodSignature("jenkins.model.HistoricalBuild", "getFullDisplayName")
     ));
 
     @Test public void sanity() throws Exception {