[jira] [Commented] (KARAF-4496) UserPrincipal lookup in the JAAS' BackingEngine

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on KARAF-4496:
---------------------------------------

jbonofre opened a new pull request #441: [KARAF-4496] Add lookupUser(username) method in JAAS backing engines
URL: https://github.com/apache/karaf/pull/441
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> UserPrincipal lookup in the JAAS' BackingEngine
> -----------------------------------------------
>
>                 Key: KARAF-4496
>                 URL: https://issues.apache.org/jira/browse/KARAF-4496
>             Project: Karaf
>          Issue Type: Improvement
>          Components: karaf-security
>            Reporter: Yurii Rashkovskii
>            Assignee: Jean-Baptiste Onofré
>            Priority: Minor
>              Labels: jaas
>             Fix For: 4.2.0, 4.1.5
>
>
> I have a suggestion for improving org.apache.karaf.jaas.modules.BackingEngine
> Currently, the only way to lookup an individual user (as far as I can tell) is to iterate through the result of BackingEngine#listUsers() and find the required user. This is fine if the number of users is reasonably low. However, when JAAS infrastructure is used for public systems (say, a SaaS), it quickly becomes a bottleneck.
> I suggest adding something like `public UserPrincipal lookupUser(String name)` to avoid this problem (also, it should return `null` if no such user found , or throw an exception)
> I am happy to contribute a PR promptly if this change will be approved.
> Thoughts?



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