URL: https://github.com/SSSD/sssd/pull/5262 Title: #5262: DN with white spaces
alexey-tikhonov commented: """ > Thanks, @alexey-tikhonov, which solution would you prefer? If I understood correctly, your approach is to use the same DN string as a table key everywhere instead of attempt to unify (transform to single state) two DN strings (with help of trimming/sanitization). If I got it right, then in general I like your approach more. But I must admit I don't understand sdap* code well enough to: - verify if you spotted all the places where this "new" attribute (SYSDB_DN_FOR_MEMBER_HASH_TABLE) must be stored/updated and read to be used as a key; - to estimate performance hit (shouldn't be dramatic though, because it's just one additional attr to store, and should have no impact during lookup; but I can't compare performance-wise with Tomas' patch). """ See the full comment at https://github.com/SSSD/sssd/pull/5262#issuecomment-682111564
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
