You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by at...@apache.org on 2007/12/19 13:38:01 UTC

svn commit: r605528 - /portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java

Author: ate
Date: Wed Dec 19 04:38:01 2007
New Revision: 605528

URL: http://svn.apache.org/viewvc?rev=605528&view=rev
Log:
Fix for JS2-833: associations between portal and remote principal are not removed
See: https://issues.apache.org/jira/browse/JS2-833

Modified:
    portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java

Modified: portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java
URL: http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java?rev=605528&r1=605527&r2=605528&view=diff
==============================================================================
--- portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java (original)
+++ portals/jetspeed-2/trunk/components/jetspeed-sso/src/main/java/org/apache/jetspeed/sso/impl/PersistenceBrokerSSOProvider.java Wed Dec 19 04:38:01 2007
@@ -622,6 +622,8 @@
 		catch (Exception e)
         {
         	e.printStackTrace();
+            // current OJB model implementation isn't 100% correct, make sure no stale/broken state is left behind
+            mapSite.remove(site);
            throw new SSOException(SSOException.FAILED_STORING_SITE_INFO_IN_DB + e.toString() );
         }
 								
@@ -634,6 +636,11 @@
          {
          	e.printStackTrace();
             throw new SSOException(SSOException.FAILED_STORING_SITE_INFO_IN_DB + e.toString() );
+         }
+         finally
+         {
+             // current OJB model implementation isn't 100% correct, make sure no stale/broken state is left behind
+             mapSite.remove(site);
          }
          
 	}



---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org