Author: ate
Date: Fri Mar 19 10:22:22 2010
New Revision: 925163
URL: http://svn.apache.org/viewvc?rev=925163&view=rev
Log:
JS2-1136: Cleanup and strengthening the Security Entity/LDAP mapping
Modified:
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/ldap/dao/impl/SpringLDAPEntityDAO.java
Modified:
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/ldap/dao/impl/SpringLDAPEntityDAO.java
URL:
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/ldap/dao/impl/SpringLDAPEntityDAO.java?rev=925163&r1=925162&r2=925163&view=diff
==============================================================================
---
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/ldap/dao/impl/SpringLDAPEntityDAO.java
(original)
+++
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/ldap/dao/impl/SpringLDAPEntityDAO.java
Fri Mar 19 10:22:22 2010
@@ -411,7 +411,10 @@ public class SpringLDAPEntityDAO impleme
{
if (!entityExists(entity))
{
- throw new
SecurityException(SecurityException.PRINCIPAL_DOES_NOT_EXIST.createScoped(entity.getType(),
entity.getId()));
+ // cannot assume external security systems like LDAP which are not
solely under the control of the Portal to be 100% in sync.
+ // removal of no longer existing entity therefore should not be
considered an error.
+ return;
+ //throw new
SecurityException(SecurityException.PRINCIPAL_DOES_NOT_EXIST.createScoped(entity.getType(),
entity.getId()));
}
String internalIdStr = entity.getInternalId();
if (internalIdStr == null)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
