diff options
author | Pavel Březina <pbrezina@redhat.com> | 2012-09-17 11:11:28 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-09-17 15:03:14 +0200 |
commit | b1caacb098ae99ad65144120fdec4d0fd98ad9d5 (patch) | |
tree | 484209b909e064186b88fd12b7d74d66293cce01 /src | |
parent | bb42eab468114ac76246053a11b5de05a57acb5a (diff) | |
download | sssd-b1caacb098ae99ad65144120fdec4d0fd98ad9d5.tar.gz sssd-b1caacb098ae99ad65144120fdec4d0fd98ad9d5.tar.bz2 sssd-b1caacb098ae99ad65144120fdec4d0fd98ad9d5.zip |
Failover: use _srv_ when no primary server is defined
https://fedorahosted.org/sssd/ticket/1521
Diffstat (limited to 'src')
-rw-r--r-- | src/providers/ad/ad_common.c | 15 | ||||
-rw-r--r-- | src/providers/ipa/ipa_common.c | 13 | ||||
-rw-r--r-- | src/providers/krb5/krb5_common.c | 15 | ||||
-rw-r--r-- | src/providers/ldap/ldap_common.c | 15 |
4 files changed, 12 insertions, 46 deletions
diff --git a/src/providers/ad/ad_common.c b/src/providers/ad/ad_common.c index 2749a96e..21a7b534 100644 --- a/src/providers/ad/ad_common.c +++ b/src/providers/ad/ad_common.c @@ -280,18 +280,9 @@ ad_failover_init(TALLOC_CTX *mem_ctx, struct be_ctx *bectx, } if (!primary_servers) { - if (backup_servers) { - DEBUG(SSSDBG_TRACE_FUNC, - ("No primary servers defined but backup are present, " - "setting backup servers as primary\n")); - primary_servers = backup_servers; - backup_servers = NULL; - } else { - DEBUG(SSSDBG_TRACE_FUNC, - ("No primary or backup servers defined but backup are present, " - "setting backup servers as primary\n")); - primary_servers = BE_SRV_IDENTIFIER; - } + DEBUG(SSSDBG_CONF_SETTINGS, + ("No primary servers defined, using service discovery\n")); + primary_servers = BE_SRV_IDENTIFIER; } ret = ad_servers_init(mem_ctx, bectx, primary_servers, options, true); diff --git a/src/providers/ipa/ipa_common.c b/src/providers/ipa/ipa_common.c index fe7ac91c..db736921 100644 --- a/src/providers/ipa/ipa_common.c +++ b/src/providers/ipa/ipa_common.c @@ -932,16 +932,9 @@ int ipa_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx, } if (!primary_servers) { - if (backup_servers) { - DEBUG(SSSDBG_CONF_SETTINGS, ("Missing primary IPA server but " - "backup server given - using it as primary!\n")); - primary_servers = backup_servers; - backup_servers = NULL; - } else { - DEBUG(SSSDBG_CONF_SETTINGS, ("Missing primary and backup IPA " - "servers - using service discovery!\n")); - primary_servers = BE_SRV_IDENTIFIER; - } + DEBUG(SSSDBG_CONF_SETTINGS, + ("No primary servers defined, using service discovery\n")); + primary_servers = BE_SRV_IDENTIFIER; } ret = ipa_servers_init(ctx, service, options, primary_servers, true); diff --git a/src/providers/krb5/krb5_common.c b/src/providers/krb5/krb5_common.c index 0e96b806..006dac1c 100644 --- a/src/providers/krb5/krb5_common.c +++ b/src/providers/krb5/krb5_common.c @@ -637,18 +637,9 @@ int krb5_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx, } if (!primary_servers) { - if (backup_servers) { - DEBUG(SSSDBG_TRACE_FUNC, - ("No primary servers defined but backup are present, " - "setting backup servers as primary\n")); - primary_servers = backup_servers; - backup_servers = NULL; - } else { - DEBUG(SSSDBG_TRACE_FUNC, - ("No primary or backup servers defined, " - "using service discovery\n")); - primary_servers = BE_SRV_IDENTIFIER; - } + DEBUG(SSSDBG_CONF_SETTINGS, + ("No primary servers defined, using service discovery\n")); + primary_servers = BE_SRV_IDENTIFIER; } ret = krb5_servers_init(ctx, service, service_name, primary_servers, true); diff --git a/src/providers/ldap/ldap_common.c b/src/providers/ldap/ldap_common.c index c11d036a..f0095b54 100644 --- a/src/providers/ldap/ldap_common.c +++ b/src/providers/ldap/ldap_common.c @@ -1243,18 +1243,9 @@ int sdap_service_init(TALLOC_CTX *memctx, struct be_ctx *ctx, } if (!urls) { - if (backup_urls) { - DEBUG(SSSDBG_CONF_SETTINGS, ("Missing primary LDAP URL but " - "backup URL given - using it " - "as primary!\n")); - urls = backup_urls; - backup_urls = NULL; - } - else { - DEBUG(SSSDBG_CONF_SETTINGS, ("Missing primary and backup LDAP " - "URLs - using service discovery!\n")); - urls = BE_SRV_IDENTIFIER; - } + DEBUG(SSSDBG_CONF_SETTINGS, + ("No primary servers defined, using service discovery\n")); + urls = BE_SRV_IDENTIFIER; } ret = sdap_urls_init(ctx, service, service_name, dns_service_name, |