[
https://issues.apache.org/jira/browse/OAK-8306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16835640#comment-16835640
]
Alex Deparvu commented on OAK-8306:
-----------------------------------
[~anchela] feedback appreciated!
> Empty PrincipalProvider cache breaks membership collection
> ----------------------------------------------------------
>
> Key: OAK-8306
> URL: https://issues.apache.org/jira/browse/OAK-8306
> Project: Jackrabbit Oak
> Issue Type: Bug
> Components: core, security
> Reporter: Alex Deparvu
> Assignee: Alex Deparvu
> Priority: Major
> Attachments: OAK-8306.patch
>
>
> Regression caused by OAK-7024 [0] when I replaced {{new HashSet<Group>(1)}}
> with {{Collections.emptySet()}} forgetting that this is actually an immutable
> set.
> {noformat}
> 02.05.2019 11:50:51.663 INFO [qtp1880753633-1912]
> org.apache.sling.auth.core.impl.SlingAuthenticator handleLoginFailure: Unable
> to authenticate null: java.lang.UnsupportedOperationExcepti
> on
> at java.util.AbstractCollection.add(AbstractCollection.java:262)
> at
> org.apache.jackrabbit.oak.security.user.UserPrincipalProvider.getGroupMembership(UserPrincipalProvider.java:267)
>
> at
> org.apache.jackrabbit.oak.security.user.UserPrincipalProvider.getMembershipPrincipals(UserPrincipalProvider.java:124)
>
> at
> org.apache.jackrabbit.oak.spi.security.authentication.AbstractLoginModule.getPrincipals(AbstractLoginModule.java:498)
>
> at
> org.apache.jackrabbit.oak.security.authentication.token.TokenLoginModule.commit(TokenLoginModule.java:154)
> {noformat}
> [0]
> https://github.com/apache/jackrabbit-oak/commit/75fa2b33cc7d5447aae2ecefbf369c3b2b2c424b#diff-bca626bc7a86cf9a4435a00386f08401L322
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
