jackrabbit-oak-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Deparvu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OAK-7232) MountPermissionProvider.load can return null
Date Thu, 01 Feb 2018 14:41:00 GMT

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

Alex Deparvu commented on OAK-7232:

patch looks good. I don't like returning nulls, but I don't have a strong preference, so feel
free to apply the patch as is.

> MountPermissionProvider.load can return null
> --------------------------------------------
>                 Key: OAK-7232
>                 URL: https://issues.apache.org/jira/browse/OAK-7232
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core, security
>            Reporter: angela
>            Assignee: angela
>            Priority: Major
>         Attachments: OAK-7232.patch
> while adding missing annotations to {{MountPermissionProvider}} i noticed that the load
method is actually defined as follows on the interface:
> {code}
> /**
>      * Loads the permission entries for the given principal and path. if the given {@code
entries} is {@code null}, it
>      * will be created automatically if needed. If a {@code entries} is given, it will
reuse it and the same object is
>      * returned. If no entries can be found for the given principal or path, {@code null}
is returned.
>      *
>      * @param entries the permission entries or {@code null}
>      * @param principalName name of the principal
>      * @param path access controlled path.
>      * @return the given {@code entries}, a new collection or {@code null}
>      */
>     @CheckForNull
>     Collection<PermissionEntry> load(@Nullable Collection<PermissionEntry>
entries, @Nonnull String principalName, @Nonnull String path);
> {code}
> IMO this means that the implementation in {{MountPermissionProvider}} could return {{null}}
instead of creating an empty set.
> [~stillalex], wdyt? (proposed patch including the missing annoatations attached).

This message was sent by Atlassian JIRA

View raw message