jackrabbit-oak-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tobias Bocanegra (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OAK-4101) Consider separate external (group) principal management
Date Thu, 12 May 2016 02:08:12 GMT

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

Tobias Bocanegra commented on OAK-4101:
---------------------------------------

as far as I can tell, this is ok. I created a temporary pull request with my comments, mostly
minor stuff:
https://github.com/apache/jackrabbit-oak/pull/52

- In general, the {{...impl.principal.External*}} code could benefit from a bit more commenting
:-)
- I ran the tests and they all pass.
- I didn't test it in a real life setup.

> Consider separate external (group) principal management
> -------------------------------------------------------
>
>                 Key: OAK-4101
>                 URL: https://issues.apache.org/jira/browse/OAK-4101
>             Project: Jackrabbit Oak
>          Issue Type: Task
>          Components: auth-external
>            Reporter: angela
>            Assignee: angela
>         Attachments: OAK-4101.patch, OAK-4101_test.patch
>
>
> Given the fact that user management is delegated to an external IDP provider, we might
reconsider the current approach that attempts to synchronize user and particularly group and
their membership into the repository.
> What would left with the repository is a dedicated {{PrincipalProvider}} for external
groups (and maybe even users at a later stage), making sure that
> - the {{Subject}} is properly populated with {{Principal}} s upon login
> - access control can still be properly setup and managed in the repository for the principals
defined in the external IDP.
> the consequences would be:
> - external groups (and potentially) users would no longer made available to the default
user management implementation. alternatively: make them available as read-only stub i.e.
group-membership as defined by the IDP could no longer be changed/manipulated in the reposiotry.
> - they are however exposed as principals to assert proper authentication + authorization.
Note: any UI that properly reflects the fact that access control is being edited for principals
(and not for users/groups) would not be affected at all; others might need to be adjusted
to additionally support ac management based on the {{PrincipalManager}}
> will try to come up with a POC as soon as I find some time.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message