[keycloak-dev] Introducing KeycloakContext

[Bill Burke]

Lol!  I was thinking the same thing yesterday.  There's other things 
that need passing around too in many areas:  UserSessionModel, 
ClientSessionModel



On 3/13/2015 2:05 AM, Stian Thorgersen wrote:
> Currently we pass a lot of things around. For example:
>
>    AuthenticationManager.logout(session, realm, userSession, uriInfo, connection)
>    Flows.forms(session, realm, client, uriInfo)
>
> I propose we introduce a KeycloakContext that can hold everything. It would be injectable by RestEasy:
>
>    @Context
>    KeycloakContext context;
>
> And would look something like:
>
>    public interface KeycloakContext {
>      KeycloakSession session();
>      RealmModel realm();
>      HttpRequest request();
>      UriInfo uriInfo();
>      ClientConnection clientConnection();
>      EventBuilder event();
>    }
>
> Also, I think we should convert AuthenticationManager and TokenManager into providers so they can be obtained from the session rather than passing it around everywhere.
>
> The above changes would be introduced after 1.2.0.Beta1 and can be done incrementally!!
> _______________________________________________
> keycloak-dev mailing list
> keycloak-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/keycloak-dev
>

-- 
Bill Burke
JBoss, a division of Red Hat
http://bill.burkecentral.com