You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2015/05/22 16:24:29 UTC
directory-fortress-core git commit: FC-105 - [fortress-core] - add
private constructors
Repository: directory-fortress-core
Updated Branches:
refs/heads/master 061320606 -> 0bbf80578
FC-105 - [fortress-core] - add private constructors
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/0bbf8057
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/0bbf8057
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/0bbf8057
Branch: refs/heads/master
Commit: 0bbf805785ad9378abe10b930d7a16c50306616a
Parents: 0613206
Author: Shawn McKinney <sm...@apache.org>
Authored: Fri May 22 09:24:26 2015 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Fri May 22 09:24:26 2015 -0500
----------------------------------------------------------------------
.../org/apache/directory/fortress/core/GlobalErrIds.java | 8 ++++++++
.../java/org/apache/directory/fortress/core/GlobalIds.java | 8 ++++++++
.../java/org/apache/directory/fortress/core/cfg/Config.java | 7 +++++++
.../apache/directory/fortress/core/rbac/AdminRoleUtil.java | 7 +++++++
.../org/apache/directory/fortress/core/rbac/AdminUtil.java | 8 ++++++++
.../org/apache/directory/fortress/core/rbac/ClassUtil.java | 8 ++++++++
.../org/apache/directory/fortress/core/rbac/HierUtil.java | 8 ++++++++
.../org/apache/directory/fortress/core/rbac/RoleUtil.java | 7 +++++++
.../java/org/apache/directory/fortress/core/rbac/SDUtil.java | 8 ++++++++
.../org/apache/directory/fortress/core/rbac/UsoUtil.java | 7 +++++++
.../org/apache/directory/fortress/core/util/LogUtil.java | 7 +++++++
.../apache/directory/fortress/core/util/attr/AttrHelper.java | 7 +++++++
.../apache/directory/fortress/core/util/attr/RegExUtil.java | 7 +++++++
.../org/apache/directory/fortress/core/util/attr/VUtil.java | 7 +++++++
.../directory/fortress/core/util/crypto/EncryptUtil.java | 8 ++++++++
.../org/apache/directory/fortress/core/util/time/CUtil.java | 7 +++++++
.../org/apache/directory/fortress/core/util/time/TUtil.java | 7 +++++++
17 files changed, 126 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/GlobalErrIds.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/GlobalErrIds.java b/src/main/java/org/apache/directory/fortress/core/GlobalErrIds.java
index 0148175..a7aaf72 100755
--- a/src/main/java/org/apache/directory/fortress/core/GlobalErrIds.java
+++ b/src/main/java/org/apache/directory/fortress/core/GlobalErrIds.java
@@ -40,6 +40,14 @@ package org.apache.directory.fortress.core;
*/
public class GlobalErrIds
{
+ /**
+ * Private constructor
+ *
+ */
+ private GlobalErrIds()
+ {
+ }
+
/** Default value when we have no error */
public static final int NO_ERROR = 0;
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/GlobalIds.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/GlobalIds.java b/src/main/java/org/apache/directory/fortress/core/GlobalIds.java
index 8b3b592..44f16da 100755
--- a/src/main/java/org/apache/directory/fortress/core/GlobalIds.java
+++ b/src/main/java/org/apache/directory/fortress/core/GlobalIds.java
@@ -38,6 +38,14 @@ import org.apache.directory.fortress.core.cfg.Config;
*/
public class GlobalIds
{
+ /**
+ * Private constructor
+ *
+ */
+ private GlobalIds()
+ {
+ }
+
public static final String HOME = "HOME";
public static final String TENANT = "tenant";
private static final String ENABLE_AUDIT = "enable.audit";
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/cfg/Config.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cfg/Config.java b/src/main/java/org/apache/directory/fortress/core/cfg/Config.java
index 94d770e..34e9e8e 100755
--- a/src/main/java/org/apache/directory/fortress/core/cfg/Config.java
+++ b/src/main/java/org/apache/directory/fortress/core/cfg/Config.java
@@ -138,6 +138,13 @@ public class Config
}
}
+ /**
+ * Private constructor
+ *
+ */
+ private Config()
+ {
+ }
/**
* Gets the prop attribute as String value from the apache commons cfg component.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/AdminRoleUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/AdminRoleUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/AdminRoleUtil.java
index e04bdcd..01d9ab4 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/AdminRoleUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/AdminRoleUtil.java
@@ -74,6 +74,13 @@ public final class AdminRoleUtil
adminRoleCache = cacheMgr.getCache( "fortress.admin.roles" );
}
+ /**
+ * Private constructor
+ *
+ */
+ private AdminRoleUtil()
+ {
+ }
/**
* Used to determine if one {@link org.apache.directory.fortress.core.rbac.AdminRole} is the parent of another. This method
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/AdminUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/AdminUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/AdminUtil.java
index f20df6e..f2b1711 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/AdminUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/AdminUtil.java
@@ -38,6 +38,14 @@ final class AdminUtil
private static final String CLS_NM = AdminUtil.class.getName();
/**
+ * Private constructor
+ *
+ */
+ private AdminUtil()
+ {
+ }
+
+ /**
* Wrapper function to call {@link DelAccessMgrImpl#canAssign(org.apache.directory.fortress.core.rbac.Session, org.apache.directory.fortress.core.rbac.User, org.apache.directory.fortress.core.rbac.Role)}.
* This will determine if the user contains an AdminRole that is authorized assignment control over User-Role Assignment (URA). This adheres to the ARBAC02 functional specification for can-assign URA.
*
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/ClassUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/ClassUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/ClassUtil.java
index b4c3c32..0db7eeb 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/ClassUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/ClassUtil.java
@@ -48,6 +48,14 @@ import java.io.InputStream;
public class ClassUtil
{
/**
+ * Private constructor
+ *
+ */
+ private ClassUtil()
+ {
+ }
+
+ /**
* Given a valid class name call the default constructor through reflexion and return the reference to the caller.
* @param className contains fully qualified java class name to be instantiated. Must have a public default constructor to be successful.
* @return reference to instantiated ManagerImpl object.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/HierUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/HierUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/HierUtil.java
index 1899ece..1a44302 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/HierUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/HierUtil.java
@@ -90,6 +90,14 @@ final class HierUtil
/**
+ * Private constructor
+ *
+ */
+ private HierUtil()
+ {
+ }
+
+ /**
*
* @param contextId
* @param type
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/RoleUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/RoleUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/RoleUtil.java
index 904090b..c030a84 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/RoleUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/RoleUtil.java
@@ -74,6 +74,13 @@ public final class RoleUtil
roleCache = cacheMgr.getCache( "fortress.roles" );
}
+ /**
+ * Private constructor
+ *
+ */
+ private void RoleUtil()
+ {
+ }
/**
* Used to determine if one {@link org.apache.directory.fortress.core.rbac.Role} is the parent of another. This method
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/SDUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/SDUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/SDUtil.java
index 5333b3b..322b648 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/SDUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/SDUtil.java
@@ -73,6 +73,14 @@ final class SDUtil
}
/**
+ * Private constructor
+ *
+ */
+ private SDUtil()
+ {
+ }
+
+ /**
* This method is called by AdminMgr.assignUser and is used to validate Static Separation of Duty
* constraints when assigning a role to user.
*
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
index f0446ae..7c7f69b 100755
--- a/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
@@ -74,6 +74,13 @@ public final class UsoUtil
usoCache = cacheMgr.getCache( "fortress.uso" );
}
+ /**
+ * Private constructor
+ *
+ */
+ private UsoUtil()
+ {
+ }
/**
* Recursively traverse the {@link org.apache.directory.fortress.core.rbac.OrgUnit} graph and return all of the descendants of a given parent {@link org.apache.directory.fortress.core.rbac.OrgUnit#name}.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/LogUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/LogUtil.java b/src/main/java/org/apache/directory/fortress/core/util/LogUtil.java
index 3a41346..970a3dc 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/LogUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/LogUtil.java
@@ -33,6 +33,13 @@ public class LogUtil
{
private static final Logger LOG = LoggerFactory.getLogger( LogUtil.class.getName() );
+ /**
+ * Private constructor
+ *
+ */
+ private LogUtil()
+ {
+ }
/**
* Write a message out to the appropriate log level.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java b/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
index 513b3fa..5b84545 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
@@ -47,6 +47,13 @@ import java.util.Properties;
public class AttrHelper
{
+ /**
+ * Private constructor
+ *
+ */
+ private AttrHelper()
+ {
+ }
/**
* Perform copy on ARBAC attributes. This is used during assignment of {@link org.apache.directory.fortress.core.rbac.AdminRole} to {@link org.apache.directory.fortress.core.rbac.User}.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/attr/RegExUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/attr/RegExUtil.java b/src/main/java/org/apache/directory/fortress/core/util/attr/RegExUtil.java
index 9b92a60..72bdfd0 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/attr/RegExUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/attr/RegExUtil.java
@@ -53,6 +53,13 @@ class RegExUtil
}
}
+ /**
+ * Private constructor
+ *
+ */
+ private RegExUtil()
+ {
+ }
/**
* Perform safe text validation on character string.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/attr/VUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/attr/VUtil.java b/src/main/java/org/apache/directory/fortress/core/util/attr/VUtil.java
index ea5d67c..f884c1b 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/attr/VUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/attr/VUtil.java
@@ -88,6 +88,13 @@ public class VUtil
DATE_FORMATER.setLenient( false );
}
+ /**
+ * Private constructor
+ *
+ */
+ private VUtil()
+ {
+ }
/**
* Simple length check on orgunit that uses {@link org.apache.directory.fortress.core.GlobalIds#OU_LEN}.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/crypto/EncryptUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/crypto/EncryptUtil.java b/src/main/java/org/apache/directory/fortress/core/util/crypto/EncryptUtil.java
index 6e7808f..53b67f7 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/crypto/EncryptUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/crypto/EncryptUtil.java
@@ -38,6 +38,14 @@ public class EncryptUtil
}
/**
+ * Private constructor
+ *
+ */
+ private EncryptUtil()
+ {
+ }
+
+ /**
* This wraps {@link #encrypt(String)} method. Will return an encrypted value to standard out using System.out.println().
* It can be used for ad-hoc encryption of vars {@code fortress.properties} bound.
* @param args contains a single String to {@link #encrypt(String)}.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java b/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
index 96026a5..a75bad3 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
@@ -65,6 +65,13 @@ public class CUtil
private static List<Validator> validators;
private static final String DSDVALIDATOR = Config.getProperty( GlobalIds.DSD_VALIDATOR_PROP );
+ /**
+ * Private constructor
+ *
+ */
+ private CUtil()
+ {
+ }
/**
* Used by DAO utilities to convert from a string with comma delimited values to fortress internal format {@link Constraint}.
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0bbf8057/src/main/java/org/apache/directory/fortress/core/util/time/TUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/time/TUtil.java b/src/main/java/org/apache/directory/fortress/core/util/time/TUtil.java
index 3dd2bae..99ba640 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/time/TUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/time/TUtil.java
@@ -31,6 +31,13 @@ import java.util.GregorianCalendar;
*/
public class TUtil
{
+ /**
+ * Private constructor
+ *
+ */
+ private TUtil()
+ {
+ }
/**
* Get the curent timestamp from Java and convert to {@link Time} format.