Only create AAF Perm when needed

-patchset 2: increment version

Change-Id: Ib2f61ed903b95e4c218aaae0850d86f6f30c31af
Signed-off-by: dglFromAtt <dgl@research.att.com>
Issue-ID: DMAAP-647
Signed-off-by: dglFromAtt <dgl@research.att.com>

2 files changed, 14 insertions, 5 deletions

diff --git a/src/main/java/org/onap/dmaap/dbcapi/server/Main.java b/src/main/java/org/onap/dmaap/dbcapi/server/Main.java
index c3340e6..d8ee278 100644
--- a/src/main/java/org/onap/dmaap/dbcapi/server/Main.java
+++ b/src/main/java/org/onap/dmaap/dbcapi/server/Main.java
@@ -24,6 +24,7 @@ import java.util.Properties;
 import java.util.UUID;
 
 import org.onap.dmaap.dbcapi.authentication.ApiPerms;
+import org.onap.dmaap.dbcapi.authentication.ApiPolicy;
 import org.onap.dmaap.dbcapi.database.DatabaseClass;
 import org.onap.dmaap.dbcapi.logging.*;
 import org.onap.dmaap.dbcapi.model.Dmaap;
@@ -97,7 +98,8 @@ public class Main extends BaseLoggingClass {
 		// for fresh installs, we may come up with no dmaap name so need to have a way for Controller to talk to us
 		Singleton<Dmaap> dmaapholder = DatabaseClass.getDmaap();
 		String name = dmaapholder.get().getDmaapName();
-		if ( name == null || name.isEmpty()) {
+		ApiPolicy apiPolicy = new ApiPolicy();
+		if ( apiPolicy.getUseAuthClass() && (name == null || name.isEmpty())) {
 			ApiPerms p = new ApiPerms();
 			p.setBootMap();
 		}
diff --git a/src/main/java/org/onap/dmaap/dbcapi/service/DmaapService.java b/src/main/java/org/onap/dmaap/dbcapi/service/DmaapService.java
index e7d4e04..de87b00 100644
--- a/src/main/java/org/onap/dmaap/dbcapi/service/DmaapService.java
+++ b/src/main/java/org/onap/dmaap/dbcapi/service/DmaapService.java
@@ -36,6 +36,7 @@ import org.onap.dmaap.dbcapi.aaf.DmaapGrant;
 import org.onap.dmaap.dbcapi.aaf.DmaapPerm;
 import org.onap.dmaap.dbcapi.aaf.AafService.ServiceType;
 import org.onap.dmaap.dbcapi.authentication.ApiPerms;
+import org.onap.dmaap.dbcapi.authentication.ApiPolicy;
 import org.onap.dmaap.dbcapi.database.DatabaseClass;
 import org.onap.dmaap.dbcapi.logging.BaseLoggingClass;
 import org.onap.dmaap.dbcapi.logging.DmaapbcLogMessageEnum;
@@ -83,8 +84,11 @@ public class DmaapService  extends BaseLoggingClass  {
 			dmaapholder.update(nd);
 
 			AafService aaf = new AafService( ServiceType.AAF_Admin);
-			ApiPerms p = new ApiPerms();
-			p.setEnvMap();
+			ApiPolicy apiPolicy = new ApiPolicy();
+			if ( apiPolicy.getUseAuthClass() ) {
+				ApiPerms p = new ApiPerms();
+				p.setEnvMap();
+			}
 			boolean anythingWrong = false;
 			
 			if ( multiSite ) {
@@ -122,8 +126,11 @@ public class DmaapService  extends BaseLoggingClass  {
 		if ( ! dmaap.isStatusValid()  || ! nd.getDmaapName().equals(dmaap.getDmaapName()) || dmaap.getVersion().equals( "0") ) {
 			nd.setLastMod();
 			dmaapholder.update(nd);  //need to set this so the following perms will pick up any new vals.
-			ApiPerms p = new ApiPerms();
-			p.setEnvMap();
+			ApiPolicy apiPolicy = new ApiPolicy();
+			if ( apiPolicy.getUseAuthClass()) {
+				ApiPerms p = new ApiPerms();
+				p.setEnvMap();
+			}
 			AafService aaf = new AafService( ServiceType.AAF_Admin);
 			if ( multiSite ) {
 				anythingWrong = setTopicMgtPerms(  nd,  aaf ) || createMmaTopic();