[SSSD] [PATCHES] Drop-redundant-sysdb_ctx-parameter

Michal Židek mzidek at redhat.com
Fri Nov 8 15:41:00 UTC 2013


Hello,

these patches (made by Jakub and me) remove redundant sysdb_ctx 
parameter from sysdb API in functions that are closely bound to some 
domain (sss_domain_info already contains sysdb_ctx, so there is no need 
to pass it as a separate parameter).


NOTE:
This is the first wave of sysdb refactoring effort. Other sysdb changes 
will follow, including ticket
https://fedorahosted.org/sssd/ticket/2129
(Always store users with FQDN with hardcoded format), which is change 
that should simplify handling of 'name' attribute and avoid code 
patterns like this:

if (IS_SUBDOMAIN) {
    parse name and domain part from 'name' attribute using
    regular expression (requires initialization of names_ctx).
} else {
    'name' attribute contains only name
}

In other words, we should store data in unified manner for main domains 
as well as for subdomains and avoid the need for regular expressions 
when simply need to parse name and domain portion from the 'name' 
attribute. Alternatively we might store name and domain portions in 
separate attributes instead of one FQDN attribute.

Thanks,
Michal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PATCHES-Drop-sysdb-ctx-parameter.tar.gz
Type: application/x-gzip
Size: 64042 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20131108/eef86b41/attachment.gz>


More information about the sssd-devel mailing list