karaf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KARAF-4496) UserPrincipal lookup in the JAAS' BackingEngine
Date Wed, 07 Feb 2018 08:45:02 GMT

    [ 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)

Mime
View raw message