Skip to content

Commit 2f7d57d

Browse files
committed
If no project is given, fall back to global permission.
See https://issues.jenkins.io/browse/JENKINS-64872 for details.
1 parent 7daad72 commit 2f7d57d

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/main/java/io/jenkins/plugins/util/JenkinsFacade.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@
88

99
import org.apache.commons.lang3.StringUtils;
1010

11+
import edu.umd.cs.findbugs.annotations.CheckForNull;
12+
1113
import org.kohsuke.stapler.Stapler;
1214
import org.kohsuke.stapler.StaplerRequest;
1315
import hudson.DescriptorExtensionList;
@@ -86,7 +88,10 @@ public boolean hasPermission(final Permission permission) {
8688
*
8789
* @return {@code false} if the user doesn't have the permission
8890
*/
89-
public boolean hasPermission(final Permission permission, final Job<?, ?> project) {
91+
public boolean hasPermission(final Permission permission, @CheckForNull final Job<?, ?> project) {
92+
if (project == null) {
93+
return hasPermission(permission);
94+
}
9095
return getJenkins().getAuthorizationStrategy().getACL(project).hasPermission(permission);
9196
}
9297

0 commit comments

Comments
 (0)