[FIXED JENKINS-13526] use '@' prefix to force PAM to interpret the user/group as a group

changelog.html

@@ -63,6 +63,9 @@ Upcoming changes</a>
   <li class=bug>
     End up more gracefully if there's some problem when searching for user partipication in the build
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-13564">issue 13564</a>)
+  <li class=rfe>
+    PAM authentication supports '@group' to force interpretation as a group instead of user.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-13526">issue 13526</a>)
   <li class=rfe>
     Added a DISCOVER permission to allow anonymous users to be presented the login screen
     when accessing job URLs.

core/src/main/java/hudson/security/PAMSecurityRealm.java

@@ -104,12 +104,18 @@ public class PAMSecurityRealm extends AbstractPasswordBasedSecurityRealm {
 
     @Override
     public GroupDetails loadGroupByGroupname(final String groupname) throws UsernameNotFoundException, DataAccessException {
-        if(CLibrary.libc.getgrnam(groupname)==null)
-            throw new UsernameNotFoundException(groupname);
+        final String group;
+        if(groupname.startsWith("@")) {
+            group = groupname.substring(1);
+        } else {
+            group = groupname;
+        }
+        if(CLibrary.libc.getgrnam(group)==null)
+            throw new UsernameNotFoundException(group);
         return new GroupDetails() {
             @Override
             public String getName() {
-                return groupname;
+                return group;
             }
         };
     }

core/src/main/resources/hudson/security/PAMSecurityRealm/help.html

@@ -7,6 +7,12 @@
   This mode will also allow you to use Unix groups for authorization. For example,
   you can say "everyone in the 'developers' group will have the administrator access".
 
+  <p>
+  Unix allows an user and a group to have the same name. If you need to disambiguate,
+  you can use the '@' prefix to force the name to be interpreted as a group. For example,
+  '@dev' would mean the 'dev' group and not the 'dev' user, while 'dev' would be interpreted
+  as an user if you indeed have the user of that name.
+
   <p>
   This is done through a library called <a href="http://en.wikipedia.org/wiki/Pluggable_Authentication_Modules">PAM</a>,
   which defines its own configuration mechanism.