[
https://issues.apache.org/jira/browse/OAK-4224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15246054#comment-15246054
]
angela edited comment on OAK-4224 at 4/18/16 5:24 PM:
------------------------------------------------------
proposed patch for {{DefaultSyncContext}} plus test-case (sorry for all the
other unrelated tests in the patch) as well as a fix for
{{CustomCredentialsSupportTest}}, where the assert effectively spotted a bug in
my test-case :-)
[~tripod], may I ask you to review the proposed fix? thanks.
was (Author: anchela):
proposed patch for {{DefaultSyncContext}} plus test-case (sorry for all the
other unrelated tests in the patch) as well as a fix for
{{CustomCredentialsSupportTest}}, where the assert effectively spotted a bug in
my test-case :-)
> DefaultSyncContext.sync(ExternalIdentity) should verify IDP
> -----------------------------------------------------------
>
> Key: OAK-4224
> URL: https://issues.apache.org/jira/browse/OAK-4224
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: auth-external
> Reporter: angela
> Priority: Minor
> Attachments: OAK-4224.patch
>
>
> while writing more test for {{DefaultSyncContext}} i realized that the
> implementation of {{sync(ExternalIdentity)}} doesn't verify that the given
> external identity belongs to the same IDP than the one associated with the
> context instance.
> IMHO this would be needed and useful particularly when multiple IDPs are
> combined. also, the {{DefaultSyncContext}} is a public exposed class, I
> would prefer if it would guard against mixing up sync of external identities
> from different sources.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
