URL: https://github.com/freeipa/freeipa/pull/535
Title: #535: add whoami command

stlaz commented:
"""
@abbra Thank you for the changes, the patch seems fine now. I tested the 
user/service/host scenarios and it worked fine. I couldn't test idviews since 
trusts are broken now but I assume it should work fine as well.
If you could only apply the following patch 
https://transfer.sh/IA7Ic/0001-improve-one-more-dict.patch which improves the 
last `dict()` behavior then I'll bless this patch :)
We may want to add some tests later so I will propose to leave the ticket open.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/535#issuecomment-285336020
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to