Github user pvillard31 commented on the issue:
    It looks OK to me, I'd just suggest allowing EL on username/password. Will 
probably be useful with the Registry stuff. I know sensitive properties are not 
handled by the registry work yet, but that could be in the future.
    Before giving a final +1, I'd appreciate another opinion on L&N aspects but 
it LGTM. Thanks for all your work @mans2singh 


Reply via email to