[java-identity-provider COMMIT] /trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSe...

[noreply at shibboleth.net]

Author: scantor
Date: Mon Oct 28 20:31:12 2013
New Revision: 4903

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=4903&view=rev
Log:
Take out unneeded TODO.

Modified:
    trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSessionManager.java

Modified: trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSessionManager.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSessionManager.java?rev=4903&r1=4902&r2=4903&view=diff
==============================================================================
--- trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSessionManager.java (original)
+++ trunk/idp-session-impl/src/main/java/net/shibboleth/idp/session/impl/StorageBackedSessionManager.java Mon Oct 28 20:31:12 2013
@@ -545,7 +545,7 @@
                     if (cookies != null) {
                         for (Cookie cookie : cookies) {
                             if (cookieName.equals(cookie.getName())) {
-                                IdPSession session = lookupByCookie(cookie);
+                                IdPSession session = lookupBySessionId(cookie.getValue());
                                 if (session != null) {
                                     return ImmutableList.of(session);
                                 }
@@ -664,21 +664,6 @@
             }
         }
     }
-    
-    /**
-     * Performs a lookup and deserializes a record based on the session identified by a cookie.
-     * 
-     * @param cookie the cookie to recover the session or session ID from
-     * @return  the IdPSession object, or null
-     * @throws ResolverException if an error occurs during lookup
-     */
-    @Nullable private IdPSession lookupByCookie(@Nonnull final Cookie cookie) throws ResolverException {
-        log.debug("Performing primary lookup on session identified by cookie");
-        
-        // TODO: handle client-side storage case
-        
-        return lookupBySessionId(cookie.getValue());
-    }
 
     /**
      * Performs a lookup and deserializes a record based on session ID.