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

Larry McCay commented on KNOX-217:
----------------------------------

I would like to see this broken up into a couple different pieces rather than 
all being in the setSystemPassword method.

* The workaround to look for the system property should not leak into your 
actual group realm code - it should be relegated to the provisioning of the 
providers part of the relevant contributor.
* The lookup of an aliased password may need to be done for other properties - 
as you have indicated - and should be made available as a common utility method 
for the shiro provider. Adding it to the AliasService made sense until we 
determined that shiro config can't deal with dollar signs.

> enhance KnoxLdapGroupRealm to accept password alias in place of plain text 
> password
> -----------------------------------------------------------------------------------
>
>                 Key: KNOX-217
>                 URL: https://issues.apache.org/jira/browse/KNOX-217
>             Project: Apache Knox
>          Issue Type: Sub-task
>          Components: Server
>    Affects Versions: 0.4.0
>            Reporter: Dilli Arumugam
>            Assignee: Dilli Arumugam
>             Fix For: 0.4.0
>
>         Attachments: KNOX-217.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to