Merge pull request #167 from pdowler/master

ACIdentityManager: bug fix to augment when only numeric principal
opencadc · Feb 28, 2024 · 2333e0f · 2333e0f
2 parents 14ab5a6 + 16f4c81
commit 2333e0f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cadc-access-control-identity/build.gradle b/cadc-access-control-identity/build.gradle
@@ -12,7 +12,7 @@ repositories {
 sourceCompatibility = 1.8
 group = 'org.opencadc'
 
-version = '1.2.3'
+version = '1.2.4'
 
 description = 'OpenCADC IdentityManager plugin library'
 def git_url = 'https://github.com/opencadc/ac'

diff --git a/cadc-access-control-identity/src/main/java/ca/nrc/cadc/ac/ACIdentityManager.java b/cadc-access-control-identity/src/main/java/ca/nrc/cadc/ac/ACIdentityManager.java
@@ -142,7 +142,7 @@ public Subject augment(final Subject subject) {
         }
 
         NumericPrincipal np = getNumericPrincipal(subject);
-        boolean needAugment = np == null;
+        boolean needAugment = (np == null || subject.getPrincipals().size() == 1);
 
         if (requireCompletePosixPrincipal) {
             PosixPrincipal pp = getPosixPrincipal(subject);