the attached patches add support for using flat name in the FQDN output
The problem with this RFE was that the format is a printf-like format
specifier and backwards compatibility must be maintained. I used an idea
that originally came from Sumit to add a new expansion to the existing
name and domain ones. But in order to avoid compile-time warnings, I had
to make sure the format specifier and the arguments match 1:1 -- so the
code adds the unused format specifier and defaults the unused arguments
[PATCH 1/3] Add utility functions for formatting fully-qualified names
Instead of using printf-like functions directly, provide two wrappers
that would encapsulate formatting the fully-qualified names. No
functional change is present in this patch.
[PATCH 2/3] Check the validity of FQname format prior to using it
Adds a sanity check of the fqname pattern. Fails if the username pattern
is not specified at all and warns if the domain pattern is not
[PATCH 3/3] Allow flat name in the FQname format
Adds another expansion in the printf format that allows the user to use
the domain flat name in the format. Includes a unit test.
Pushed to master under the one-liner rule.
When I was rebasing my patches on top of Lukas' change to tests, I
linked with his new library to make sure the tests pass before pushing
but I forgot to commit this change. Sorry for the churn.
This patch removes empty directories after tests run.
Files common_dom.c and common_tev.c were moved to libsss_test_common.la.
Empty directory tests_path was removed in function test_dom_suite_cleanup.
Function test_dom_suite_cleanup was reused in tests.
I fond these small issues while working on the GC lookups. I'm sending
them separately b/c they can be reviewed without any knowledge about GC
and do not break any existing functionality.
[PATCH 1/4] IPA: Check for ENOMEM
Just an allocation check.
[PATCH 2/4] Remove unneeded comment
Not needed anymore I think
[PATCH 3/4] FO: Fix setting status of duplicate
The fo_set_port_status() function was performing its own duplicate
detection which is wrong because it duplicates existing code and doesn't
take the private user_data structure into account.
[PATCH 4/4] AD dyndns: extract the host name from URI
Because the AD provider might connect to LDAP on different ports
depending on whether it is a Global Catalog or pure LDAP, we need to
extract the hostname using LDAP function calls, not just grab the server
name part from the URI.
When working on the GC lookups I found out that the recent dyndns
patches were scheduling the periodic updates even if dyndns_update was
set to false. The attached patch fixes that. Sorry for the breakage.
I'd like to add tags to the domains in sssd.conf from realmd. sssd
ignores unknown attributes, but SSSDConfig needs to be told about them.
Attached is a patch which adds such an attribute.
This is related to: https://bugzilla.redhat.com/show_bug.cgi?id=966650