You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2014/10/21 01:06:33 UTC
[16/50] git commit: FC-139 - AcceleratorDAO CheckAccess sending empty
objectId in request
FC-139 - AcceleratorDAO CheckAccess sending empty objectId in request
Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/5f07c581
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/5f07c581
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/5f07c581
Branch: refs/heads/master
Commit: 5f07c5817d523e7d816c7e212d75f65248169ee2
Parents: 7f33959
Author: Shawn McKinney <sh...@jts.us>
Authored: Mon Jun 30 07:51:30 2014 -0500
Committer: Shawn McKinney <sh...@jts.us>
Committed: Mon Jun 30 07:51:30 2014 -0500
----------------------------------------------------------------------
.../org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/5f07c581/src/main/java/org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java b/src/main/java/org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java
index 187ed1e..f6f2f03 100644
--- a/src/main/java/org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java
+++ b/src/main/java/org/openldap/fortress/rbac/dao/apache/AcceleratorDAO.java
@@ -160,7 +160,11 @@ public final class AcceleratorDAO extends ApacheDsDataProvider implements org.op
RbacCheckAccessRequest rbacCheckAccessRequest = new RbacCheckAccessRequestImpl();
rbacCheckAccessRequest.setSessionId( session.getSessionId() );
rbacCheckAccessRequest.setObject( perm.getObjName() );
- rbacCheckAccessRequest.setObjectId( perm.getObjId() );
+ // objectId is optional
+ if(VUtil.isNotNullOrEmpty( perm.getObjId()))
+ {
+ rbacCheckAccessRequest.setObjectId( perm.getObjId() );
+ }
rbacCheckAccessRequest.setOperation( perm.getOpName() );
// Send the request
RbacCheckAccessResponse rbacCheckAccessResponse = ( RbacCheckAccessResponse ) ld.extended(