This is an automated email from the ASF dual-hosted git repository.

jgallimore pushed a commit to branch tomee-1.7.x
in repository https://gitbox.apache.org/repos/asf/tomee.git

commit ec686be5c8dc950851a0969907ca8ba38f9be456
Author: Jonathan Gallimore <[email protected]>
AuthorDate: Wed May 22 11:15:59 2019 +0100

    For review; We shouldn't use javax.security.jacc.policy.provider here, 
because the System policy will already be set. We need to ensure that our 
policy provider is used, which will delegate to the system policy as appropriate
---
 .../java/org/apache/openejb/core/security/AbstractSecurityService.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/container/openejb-core/src/main/java/org/apache/openejb/core/security/AbstractSecurityService.java
 
b/container/openejb-core/src/main/java/org/apache/openejb/core/security/AbstractSecurityService.java
index f40bf09..42150c9 100644
--- 
a/container/openejb-core/src/main/java/org/apache/openejb/core/security/AbstractSecurityService.java
+++ 
b/container/openejb-core/src/main/java/org/apache/openejb/core/security/AbstractSecurityService.java
@@ -380,7 +380,7 @@ public abstract class AbstractSecurityService implements 
SecurityService<UUID>,
             Thread.currentThread().setContextClassLoader(contextClassLoader);
         }
 
-        final String policyProvider = 
SystemInstance.get().getOptions().get("javax.security.jacc.policy.provider", 
JaccProvider.Policy.class.getName());
+        final String policyProvider = JaccProvider.Policy.class.getName();
         try {
             final ClassLoader classLoader = 
Thread.currentThread().getContextClassLoader();
             final Class policyClass = Class.forName(policyProvider, true, 
classLoader);

Reply via email to