[JENKINS-18978] - Fixed a groups list for slaves.

Resolves: https://issues.jenkins-ci.org/browse/JENKINS-18978 Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>
jenkinsci · Dec 7, 2013 · 6c2e145 · 6c2e145
1 parent 216470b
commit 6c2e145
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/...ain/java/com/michelin/cio/hudson/plugins/rolestrategy/RoleBasedAuthorizationStrategy.java b/...ain/java/com/michelin/cio/hudson/plugins/rolestrategy/RoleBasedAuthorizationStrategy.java
@@ -41,7 +41,10 @@
 import hudson.model.Hudson;
 import hudson.model.Item;
 import hudson.model.Job;
+import hudson.model.Project;
+import hudson.model.Run;
 import hudson.model.View;
+import hudson.scm.SCM;
 import hudson.security.ACL;
 import hudson.security.AccessControlled;
 import hudson.security.AuthorizationStrategy;
@@ -574,7 +577,11 @@ else if (type.equals(SLAVE)) {
             groups.remove(PermissionGroup.get(Permission.class));
             groups.remove(PermissionGroup.get(Hudson.class));
             groups.remove(PermissionGroup.get(View.class));
-            groups.remove(PermissionGroup.get(Job.class));
+
+            // Project, SCM and Run permissions 
+            groups.remove(PermissionGroup.get(Item.class));
+            groups.remove(PermissionGroup.get(SCM.class));
+            groups.remove(PermissionGroup.get(Run.class));
         }
         else {
             groups = null;