URL: https://github.com/freeipa/freeipa/pull/573
Title: #573: Provide centralized management of user short name resolution

HonzaCholasta commented:
I would rather avoid the refactoring in 4.5 - this is fragile code you are 
touching and I'm afraid it might break in some cases (think different client / 
server version combinations, thin client vs fat client, etc.).

As for the edge case values, IMO we should allow `:` without complaining as a 
special case to support "no domains in the list" configuration, and otherwise 
require known domain names (like in `certmaprule-add`).

See the full comment at 
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to