[ 
https://issues.apache.org/jira/browse/SLING-1534?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12873125#action_12873125
 ] 

Felix Meschberger commented on SLING-1534:
------------------------------------------

Patch looks good to me.

> +        // TODO - figure out if this can be change to use authInfo instead 
> of the session
>         request.setAttribute(HttpContext.REMOTE_USER, session.getUserID());

How about adding a ResourceResolver.getUserID() (or similar) method, which may 
in the case of JCR be implemented in terms of Session.getUserID() ?

> switch SlingAuthenticator to use ResourceResolverFactory API rather than JCR 
> directory
> --------------------------------------------------------------------------------------
>
>                 Key: SLING-1534
>                 URL: https://issues.apache.org/jira/browse/SLING-1534
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>            Reporter: Justin Edelson
>             Fix For: Commons Auth 1.0.0
>
>         Attachments: SLING-1534.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to