You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openaz.apache.org by an...@apache.org on 2015/07/18 05:33:37 UTC
incubator-openaz git commit: Updated to take care of duplicate
loading of config file and fixed tests
Repository: incubator-openaz
Updated Branches:
refs/heads/master 8a2d2cf68 -> 9cc406d19
Updated to take care of duplicate loading of config file and fixed tests
Project: http://git-wip-us.apache.org/repos/asf/incubator-openaz/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-openaz/commit/9cc406d1
Tree: http://git-wip-us.apache.org/repos/asf/incubator-openaz/tree/9cc406d1
Diff: http://git-wip-us.apache.org/repos/asf/incubator-openaz/diff/9cc406d1
Branch: refs/heads/master
Commit: 9cc406d19e2e690baeec25a66abd274aadafe3dd
Parents: 8a2d2cf
Author: Ajith Nair <an...@apache.org>
Authored: Thu Jul 16 22:59:41 2015 -0400
Committer: Ajith Nair <an...@apache.org>
Committed: Thu Jul 16 22:59:41 2015 -0400
----------------------------------------------------------------------
.gitignore | 1 +
.../org/apache/openaz/pepapi/std/PepUtils.java | 28 +++++++++++++-------
.../openaz/pepapi/std/StdPepAgentFactory.java | 3 +--
.../apache/openaz/pepapi/std/test/TestAPI.java | 2 +-
.../openaz/pepapi/std/test/TestDataTypes.java | 2 +-
.../openaz/pepapi/std/test/TestMapper.java | 2 +-
.../properties/testapi.xacml.properties | 2 +-
.../properties/testdatatypes.xacml.properties | 2 +-
.../properties/testmapper.xacml.properties | 2 +-
9 files changed, 27 insertions(+), 17 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/.gitignore
----------------------------------------------------------------------
diff --git a/.gitignore b/.gitignore
index 2db461e..4ebc016 100644
--- a/.gitignore
+++ b/.gitignore
@@ -2,6 +2,7 @@ target
.project
.classpath
.settings
+.idea
*.log
*.iml
*.ipr
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/PepUtils.java
----------------------------------------------------------------------
diff --git a/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/PepUtils.java b/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/PepUtils.java
index 40dbe20..69dc0e1 100644
--- a/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/PepUtils.java
+++ b/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/PepUtils.java
@@ -24,7 +24,9 @@ import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import java.io.*;
+import java.util.HashSet;
import java.util.Properties;
+import java.util.Set;
/**
*
@@ -64,7 +66,7 @@ public class PepUtils {
// Try the location as a file first.
File file = new File(propertyFile);
- InputStream in;
+ InputStream in = null;
if (file.exists() && file.canRead()) {
if (!file.isAbsolute()) {
file = file.getAbsoluteFile();
@@ -72,15 +74,23 @@ public class PepUtils {
try {
in = new FileInputStream(file);
} catch (FileNotFoundException e) {
- logger.info(propertyFile + " is not a file.");
+ logger.error("Error while accessing file: " + propertyFile);
+ throw new IllegalArgumentException(e);
+ }
+ } else {
+ Set<ClassLoader> classLoaders = new HashSet<>();
+ classLoaders.add(PepUtils.class.getClassLoader());
+ classLoaders.add(Thread.currentThread().getContextClassLoader());
+ for(ClassLoader classLoader: classLoaders) {
+ in = classLoader.getResourceAsStream(propertyFile);
+ if(in != null) {
+ break;
+ }
+ }
+ if(in == null) {
+ logger.error("Invalid classpath or file location: " + propertyFile);
+ throw new IllegalArgumentException("Invalid classpath or file location: " + propertyFile);
}
- }
-
- in = PepUtils.class.getResourceAsStream(propertyFile);
-
- if (in == null) {
- logger.error("Invalid classpath of file location: " + propertyFile);
- throw new IllegalArgumentException("Invalid classpath or file location: " + propertyFile);
}
try {
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/StdPepAgentFactory.java
----------------------------------------------------------------------
diff --git a/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/StdPepAgentFactory.java b/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/StdPepAgentFactory.java
index e6ad7ba..1806e88 100644
--- a/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/StdPepAgentFactory.java
+++ b/openaz-pep/src/main/java/org/apache/openaz/pepapi/std/StdPepAgentFactory.java
@@ -47,8 +47,7 @@ public class StdPepAgentFactory implements PepAgentFactory {
this.xacmlProperties = properties;
this.pepConfig = new StdPepConfig(properties);
try {
- // FIXME: Error when invoking newInstance() with properties.
- pdpEngineFactory = PDPEngineFactory.newInstance();
+ pdpEngineFactory = PDPEngineFactory.newInstance(properties);
} catch (FactoryException e) {
throw new PepException(e);
}
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestAPI.java
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestAPI.java b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestAPI.java
index f60f91e..d6c96b4 100644
--- a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestAPI.java
+++ b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestAPI.java
@@ -35,7 +35,7 @@ public class TestAPI {
@Before
public void setup() {
- pepAgentFactory = new StdPepAgentFactory("/properties/testapi.xacml.properties");
+ pepAgentFactory = new StdPepAgentFactory("properties/testapi.xacml.properties");
}
/**
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestDataTypes.java
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestDataTypes.java b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestDataTypes.java
index 632a8fe..4276ea6 100644
--- a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestDataTypes.java
+++ b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestDataTypes.java
@@ -40,7 +40,7 @@ public class TestDataTypes {
* System.setProperty("xacml.properties" ,
* getClass().getClassLoader().getResource("properties/testdatatypes.xacml.properties").getPath());
*/
- pepAgentFactory = new StdPepAgentFactory("/properties/testdatatypes.xacml.properties");
+ pepAgentFactory = new StdPepAgentFactory("properties/testdatatypes.xacml.properties");
}
/**
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestMapper.java
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestMapper.java b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestMapper.java
index 035d644..72a6f88 100644
--- a/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestMapper.java
+++ b/openaz-pep/src/test/java/org/apache/openaz/pepapi/std/test/TestMapper.java
@@ -43,7 +43,7 @@ public class TestMapper {
@Before
public void setup() {
- this.pepAgentFactory = new StdPepAgentFactory("/properties/testmapper.xacml.properties");
+ this.pepAgentFactory = new StdPepAgentFactory("properties/testmapper.xacml.properties");
}
/**
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/resources/properties/testapi.xacml.properties
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/resources/properties/testapi.xacml.properties b/openaz-pep/src/test/resources/properties/testapi.xacml.properties
index 75c84a3..7ba87d7 100755
--- a/openaz-pep/src/test/resources/properties/testapi.xacml.properties
+++ b/openaz-pep/src/test/resources/properties/testapi.xacml.properties
@@ -2,7 +2,7 @@
# Standard API Factories
#
xacml.dataTypeFactory=org.apache.openaz.xacml.std.StdDataTypeFactory
-xacml.pdpEngineFactory=org.apache.openaz.xacmlatt.pdp.ATTPDPEngineFactory
+xacml.pdpEngineFactory=org.apache.openaz.xacml.pdp.OpenAZPDPEngineFactory
xacml.pepEngineFactory=org.apache.openaz.xacml.std.pep.StdEngineFactory
xacml.pipFinderFactory=org.apache.openaz.xacml.std.pip.StdPIPFinderFactory
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/resources/properties/testdatatypes.xacml.properties
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/resources/properties/testdatatypes.xacml.properties b/openaz-pep/src/test/resources/properties/testdatatypes.xacml.properties
index ee83c87..c4e8ef5 100755
--- a/openaz-pep/src/test/resources/properties/testdatatypes.xacml.properties
+++ b/openaz-pep/src/test/resources/properties/testdatatypes.xacml.properties
@@ -2,7 +2,7 @@
# Standard API Factories
#
xacml.dataTypeFactory=org.apache.openaz.xacml.std.StdDataTypeFactory
-xacml.pdpEngineFactory=org.apache.openaz.xacmlatt.pdp.ATTPDPEngineFactory
+xacml.pdpEngineFactory=org.apache.openaz.xacml.pdp.OpenAZPDPEngineFactory
xacml.pepEngineFactory=org.apache.openaz.xacml.std.pep.StdEngineFactory
xacml.pipFinderFactory=org.apache.openaz.xacml.std.pip.StdPIPFinderFactory
http://git-wip-us.apache.org/repos/asf/incubator-openaz/blob/9cc406d1/openaz-pep/src/test/resources/properties/testmapper.xacml.properties
----------------------------------------------------------------------
diff --git a/openaz-pep/src/test/resources/properties/testmapper.xacml.properties b/openaz-pep/src/test/resources/properties/testmapper.xacml.properties
index 676b748..22c67a3 100755
--- a/openaz-pep/src/test/resources/properties/testmapper.xacml.properties
+++ b/openaz-pep/src/test/resources/properties/testmapper.xacml.properties
@@ -2,7 +2,7 @@
# Standard API Factories
#
xacml.dataTypeFactory=org.apache.openaz.xacml.std.StdDataTypeFactory
-xacml.pdpEngineFactory=org.apache.openaz.xacmlatt.pdp.ATTPDPEngineFactory
+xacml.pdpEngineFactory=org.apache.openaz.xacml.pdp.OpenAZPDPEngineFactory
xacml.pepEngineFactory=org.apache.openaz.xacml.std.pep.StdEngineFactory
xacml.pipFinderFactory=org.apache.openaz.xacml.std.pip.StdPIPFinderFactory