[keycloak-dev] session.userLocalStorage() needs to be fronted by cache

[Bill Burke]

2.4.1


On 11/21/16 1:42 PM, Stian Thorgersen wrote:
> We're about to release 2.4.0.CR1. When will this be ready?
>
> On 21 November 2016 at 17:18, Bill Burke <bburke at redhat.com 
> <mailto:bburke at redhat.com>> wrote:
>
>     I think I made a mistake when User Storage Providers are importing
>     into
>     local cache.  Currently KeycloakSession.userLocalStorage() does
>     not have
>     a cache in front of it.  The LDAP and Kerberos providers call this to
>     determine if the user has been imported or not.  The thing is, the
>     user
>     may already be cached and I think there is a possibility of
>     updating the
>     user (on demand resync) and getting stale cache entries. Also, we
>     don't
>     want a database hit every time there is validation happening.
>
>     So, I'm going to figure out a way to have the cache front the
>     userLocalStorage() method too like we do for userStorage(). This will
>     require some refactoring of UserCacheSession.  Not sure if that will
>     conflict with Marek's work.
>
>     Bill
>
>     _______________________________________________
>     keycloak-dev mailing list
>     keycloak-dev at lists.jboss.org <mailto:keycloak-dev at lists.jboss.org>
>     https://lists.jboss.org/mailman/listinfo/keycloak-dev
>     <https://lists.jboss.org/mailman/listinfo/keycloak-dev>
>
>