You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@sling.apache.org by fm...@apache.org on 2010/01/31 11:44:35 UTC

svn commit: r904996 - /sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java

Author: fmeschbe
Date: Sun Jan 31 10:44:35 2010
New Revision: 904996

URL: http://svn.apache.org/viewvc?rev=904996&view=rev
Log:
Fix remove method to return null *if* the key is AUTH_TYPE (see comments in SLING-1314 for details).

Modified:
    sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java

Modified: sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java?rev=904996&r1=904995&r2=904996&view=diff
==============================================================================
--- sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java (original)
+++ sling/trunk/bundles/commons/auth/src/main/java/org/apache/sling/commons/auth/spi/AuthenticationInfo.java Sun Jan 31 10:44:35 2010
@@ -372,8 +372,8 @@
     @Override
     public Object remove(Object key) {
 
-        // don't return the auth type from the map
-        if (!AUTH_TYPE.equals(key)) {
+        // don't remove the auth type from the map
+        if (AUTH_TYPE.equals(key)) {
             return null;
         }