You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-issues@jackrabbit.apache.org by "angela (JIRA)" <ji...@apache.org> on 2018/05/04 07:15:00 UTC

[jira] [Commented] (OAK-7470) Remove Usage of ImmutableTree and AbstractTree in Security Code

    [ https://issues.apache.org/jira/browse/OAK-7470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16463473#comment-16463473 ] 

angela commented on OAK-7470:
-----------------------------

[~stillalex], [~rombert], [~mduerig], [~mreutegg], proposed patch attached. Since the {{TreeProvider}} is exported API, I would appreciate your feedback on this. Ultimately this changes falls under the m12n umbrella as there is currently no way to get back the {{NodeState}} from the read-only trees obtained through the {{TreeProvider}}.

> Remove Usage of ImmutableTree and AbstractTree in Security Code
> ---------------------------------------------------------------
>
>                 Key: OAK-7470
>                 URL: https://issues.apache.org/jira/browse/OAK-7470
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: authorization-cug, core, security-spi
>            Reporter: angela
>            Assignee: angela
>            Priority: Major
>              Labels: m12n
>         Attachments: OAK-7470-tests.patch, OAK-7470.patch
>
>
> With a minor extension to {{TreeProvider}} we would be able to get rid of the direct casting to implementation details like {{ImmutableTree}} and {{AbstractTree}} altogether.
> [~stillalex], patch for review will follow right away.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)