<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Just to add to the previous changes, I've also made a change to the
    IdentityStore API so that each method includes an
    IdentityStoreInvocationContext parameter.&nbsp; This API was first
    introduced by Bolek in PicketLink 1.x [1] and offers a robust method
    of passing context-specific state to individual IdentityStore
    invocations.&nbsp; <br>
    <br>
    I can't think of any more elegant design that addresses this
    requirement or improves on the work Bolek has already done, so
    beyond any minor tweaks to the API I think we should stick with the
    motto "if it ain't broke, don't fix it".&nbsp; <br>
    <br>
    I've gone ahead and updated both (JPA and LDAP) IdentityStore
    implementations to include these parameters.&nbsp; Along the way though,
    it seems I've broken a couple of the LDAP tests; Pedro or Anil,
    could you please take a look at these as my LDAP is a little rusty -
    I've attached the stack trace output from the test run.<br>
    <br>
    [1]
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    <a
href="https://github.com/picketlink/picketlink-idm/tree/1.4/picketlink-idm-spi/src/main/java/org/picketlink/idm/spi/store">https://github.com/picketlink/picketlink-idm/tree/1.4/picketlink-idm-spi/src/main/java/org/picketlink/idm/spi/store</a>
  </body>
</html>