From 4a3962dec1da8d43e0db225d42fa4d57e338e1d6 Mon Sep 17 00:00:00 2001 From: Sean Hassan Date: Thu, 17 Sep 2020 15:54:19 -0500 Subject: Prepare for release 2.1.18 Issue-ID: AAF-1202 Change-Id: I138ff4d94ed17a5b2de24fa95daf7d2a38eb84c0 Signed-off-by: Sean Hassan --- pom.xml | 4 +- shiro-osgi-bundle/pom.xml | 2 +- shiro/pom.xml | 2 +- .../onap/aaf/cadi/shiro/AAFAuthorizationInfo.java | 54 ++++++++++++++++------ .../java/org/onap/aaf/cadi/shiro/AAFRealm.java | 12 +++-- sidecar/fproxy/pom.xml | 2 +- sidecar/pom.xml | 2 +- sidecar/rproxy/pom.xml | 2 +- sidecar/tproxy-config/pom.xml | 2 +- version.properties | 2 +- 10 files changed, 57 insertions(+), 27 deletions(-) diff --git a/pom.xml b/pom.xml index b896c4e..8b0096f 100644 --- a/pom.xml +++ b/pom.xml @@ -22,7 +22,7 @@ 4.0.0 org.onap.aaf.cadi parent - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT CADI Plugins Parent pom @@ -34,7 +34,7 @@ - 2.7.1 + 2.7.2 UTF-8 diff --git a/shiro-osgi-bundle/pom.xml b/shiro-osgi-bundle/pom.xml index 039b79e..531ee90 100644 --- a/shiro-osgi-bundle/pom.xml +++ b/shiro-osgi-bundle/pom.xml @@ -22,7 +22,7 @@ org.onap.aaf.cadi parent - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT .. diff --git a/shiro/pom.xml b/shiro/pom.xml index 4810acb..fada340 100644 --- a/shiro/pom.xml +++ b/shiro/pom.xml @@ -22,7 +22,7 @@ org.onap.aaf.cadi parent - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT .. diff --git a/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFAuthorizationInfo.java b/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFAuthorizationInfo.java index ac62778..247683a 100644 --- a/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFAuthorizationInfo.java +++ b/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFAuthorizationInfo.java @@ -42,6 +42,7 @@ public class AAFAuthorizationInfo implements AuthorizationInfo { private Access access; private Principal bait; + private List pond; // Use these to save conversions private List cPerms; private List oPerms; @@ -53,6 +54,16 @@ public class AAFAuthorizationInfo implements AuthorizationInfo { cPerms=null; oPerms=null; sPerms=null; + pond=null; + } + + public AAFAuthorizationInfo(Access access, Principal bait, List pond) { + this.access = access; + this.bait = bait; + this.pond = pond; + oPerms=null; + sPerms=null; + cPerms=null; } public Principal principal() { @@ -64,13 +75,20 @@ public class AAFAuthorizationInfo implements AuthorizationInfo { access.log(Level.DEBUG, "AAFAuthorizationInfo.getObjectPermissions"); synchronized(bait) { if(oPerms == null) { - oPerms = new ArrayList<>(); - if(cPerms==null) { - cPerms = new ArrayList<>(); - AAFRealm.singleton.authz.fishAll(bait, cPerms); - } - for(final org.onap.aaf.cadi.Permission p : cPerms) { - oPerms.add(new AAFShiroPermission(p)); + if (pond != null) { + oPerms = new ArrayList(); + for(final org.onap.aaf.cadi.Permission p : pond) { + oPerms.add(new AAFShiroPermission(p)); + } + } else { + oPerms = new ArrayList<>(); + if (cPerms == null) { + cPerms = new ArrayList<>(); + AAFRealm.singleton.authz.fishAll(bait, cPerms); + } + for (final org.onap.aaf.cadi.Permission p : cPerms) { + oPerms.add(new AAFShiroPermission(p)); + } } } } @@ -89,13 +107,21 @@ public class AAFAuthorizationInfo implements AuthorizationInfo { access.log(Level.DEBUG,"AAFAuthorizationInfo.getStringPermissions"); synchronized(bait) { if(sPerms == null) { - sPerms = new ArrayList<>(); - if(cPerms==null) { - cPerms = new ArrayList<>(); - AAFRealm.singleton.authz.fishAll(bait,cPerms); - } - for(final org.onap.aaf.cadi.Permission p : cPerms) { - sPerms.add(p.getKey()); + if (pond != null) { + sPerms = new ArrayList(); + for(org.onap.aaf.cadi.Permission p : pond) { + sPerms.add(p.getKey().replace("|", ":")); + access.printf(Level.INFO, "%s has %s", bait.getName(), p.getKey()); + } + } else { + sPerms = new ArrayList<>(); + if (cPerms == null) { + cPerms = new ArrayList<>(); + AAFRealm.singleton.authz.fishAll(bait, cPerms); + } + for (final org.onap.aaf.cadi.Permission p : cPerms) { + sPerms.add(p.getKey()); + } } } } diff --git a/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFRealm.java b/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFRealm.java index d213b01..818ec9e 100644 --- a/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFRealm.java +++ b/shiro/src/main/java/org/onap/aaf/cadi/shiro/AAFRealm.java @@ -22,6 +22,7 @@ package org.onap.aaf.cadi.shiro; import java.io.IOException; import java.security.Principal; +import java.util.ArrayList; import java.util.Collections; import java.util.List; import java.util.Map; @@ -38,6 +39,7 @@ import org.apache.shiro.subject.PrincipalCollection; import org.onap.aaf.cadi.Access.Level; import org.onap.aaf.cadi.CadiException; import org.onap.aaf.cadi.LocatorException; +import org.onap.aaf.cadi.Permission; import org.onap.aaf.cadi.PropAccess; import org.onap.aaf.cadi.Symm; import org.onap.aaf.cadi.aaf.AAFPermission; @@ -252,7 +254,7 @@ public class AAFRealm extends AuthorizingRealm { } } catch (IOException e) { singleton.access.log(e); - } + } } String err; try { @@ -302,7 +304,9 @@ public class AAFRealm extends AuthorizingRealm { }; } } - return new AAFAuthorizationInfo(singleton.access,newBait); + List pond = new ArrayList<>(); + singleton.authz.fishAll(newBait,pond); + return new AAFAuthorizationInfo(singleton.access,bait,pond); } @Override @@ -328,7 +332,7 @@ public class AAFRealm extends AuthorizingRealm { return null; } } - +/* @Override public boolean isPermitted(PrincipalCollection principals, String permission) { logger.debug("AAFRealm.isPermitted(principals,permission)"); @@ -370,5 +374,5 @@ public class AAFRealm extends AuthorizingRealm { } return super.isPermitted(permissions, info); } - +*/ } diff --git a/sidecar/fproxy/pom.xml b/sidecar/fproxy/pom.xml index 94d6a41..eb680ba 100644 --- a/sidecar/fproxy/pom.xml +++ b/sidecar/fproxy/pom.xml @@ -24,7 +24,7 @@ org.onap.aaf.cadi.sidecar sidecar - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT diff --git a/sidecar/pom.xml b/sidecar/pom.xml index ed01273..bba5587 100644 --- a/sidecar/pom.xml +++ b/sidecar/pom.xml @@ -20,7 +20,7 @@ org.onap.aaf.cadi parent - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT .. 4.0.0 diff --git a/sidecar/rproxy/pom.xml b/sidecar/rproxy/pom.xml index 6e1789e..368c833 100644 --- a/sidecar/rproxy/pom.xml +++ b/sidecar/rproxy/pom.xml @@ -24,7 +24,7 @@ org.onap.aaf.cadi.sidecar sidecar - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT diff --git a/sidecar/tproxy-config/pom.xml b/sidecar/tproxy-config/pom.xml index 744ff1a..e1de788 100644 --- a/sidecar/tproxy-config/pom.xml +++ b/sidecar/tproxy-config/pom.xml @@ -24,7 +24,7 @@ org.onap.aaf.cadi.sidecar sidecar - 2.1.17-SNAPSHOT + 2.1.18-SNAPSHOT tproxy-config diff --git a/version.properties b/version.properties index 3be336d..c276227 100644 --- a/version.properties +++ b/version.properties @@ -28,7 +28,7 @@ # This TAG 2.1.13 is here to help remember to change this file. Keep it up to date with the following "real" entries: major=2 minor=1 -patch=17 +patch=18 base_version=${major}.${minor}.${patch} -- cgit 1.2.3-korg