|
||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||
|
||||||||
|
This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira |
||||||||
Ah yes you're right.. for some reason I was thinking of IdentityManager (probably because of the SecurityContext parameter), not IdentityStore. We should definitely cache the store instances at initialization time. We definitely should not be passing the SecurityContext to IdentityStore.setup(), because the SecurityContext is per-request and the store is shared by all requests. Agreed we should remove SecurityContext from createIdentityStore, that is a relic left over from our previous design.