Blame SOURCES/0085-DYNDNS-Don-t-use-server-cmd-in-nsupdate-by-default.patch

6cf099
From 6d1ba78ff110ee424e54025c3d823fb5c504aefa Mon Sep 17 00:00:00 2001
6cf099
From: Pavel Reichl <preichl@redhat.com>
6cf099
Date: Thu, 23 Jul 2015 04:40:03 -0400
6cf099
Subject: [PATCH 85/90] DYNDNS: Don't use server cmd in nsupdate by default
6cf099
6cf099
nsupdate command `server` should not be used for the first attempt
6cf099
to udpate DNS. It should be used only in subsequent attempts after the
6cf099
first attempt failed.
6cf099
6cf099
Resolves:
6cf099
https://fedorahosted.org/sssd/ticket/2495
6cf099
6cf099
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
6cf099
(cherry picked from commit 7c3cc1ee2914bc7b38a992c1af254fc76af5a1ad)
6cf099
---
6cf099
 src/man/sssd-ad.5.xml            | 6 +++++-
6cf099
 src/man/sssd-ipa.5.xml           | 5 +++++
6cf099
 src/providers/ldap/sdap_dyndns.c | 2 +-
6cf099
 3 files changed, 11 insertions(+), 2 deletions(-)
6cf099
6cf099
diff --git a/src/man/sssd-ad.5.xml b/src/man/sssd-ad.5.xml
6cf099
index 7ccd29794a89fa6b69b744a47da04f908efc7ef9..127e96582d71e8216db88d37a16d37d01748131d 100644
6cf099
--- a/src/man/sssd-ad.5.xml
6cf099
+++ b/src/man/sssd-ad.5.xml
6cf099
@@ -826,12 +826,16 @@ ad_gpo_map_deny = +my_pam_service
6cf099
                             server.
6cf099
                         </para>
6cf099
                         <para>
6cf099
+                            Please note that this option will be only used in
6cf099
+                            fallback attempt when previous attempt using
6cf099
+                            autodetected settings failed.
6cf099
+                        </para>
6cf099
+                        <para>
6cf099
                             Default: None (let nsupdate choose the server)
6cf099
                         </para>
6cf099
                     </listitem>
6cf099
                 </varlistentry>
6cf099
 
6cf099
-
6cf099
                 <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="include/override_homedir.xml" />
6cf099
                 <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="include/homedir_substring.xml" />
6cf099
 
6cf099
diff --git a/src/man/sssd-ipa.5.xml b/src/man/sssd-ipa.5.xml
6cf099
index 871c41607b97bd24fe5feaa282258def0fd0cc8b..be390d58dd3ce5a6ca6d5212d2c7aa176d3a74c4 100644
6cf099
--- a/src/man/sssd-ipa.5.xml
6cf099
+++ b/src/man/sssd-ipa.5.xml
6cf099
@@ -276,6 +276,11 @@
6cf099
                             server.
6cf099
                         </para>
6cf099
                         <para>
6cf099
+                            Please note that this option will be only used in
6cf099
+                            fallback attempt when previous attempt using
6cf099
+                            autodetected settings failed.
6cf099
+                        </para>
6cf099
+                        <para>
6cf099
                             Default: None (let nsupdate choose the server)
6cf099
                         </para>
6cf099
                     </listitem>
6cf099
diff --git a/src/providers/ldap/sdap_dyndns.c b/src/providers/ldap/sdap_dyndns.c
6cf099
index 01f4f17226f1b7dd417699403b425c571b780c3a..0f0e63ee2259d017c3e94afca5f3148f4fc2ce04 100644
6cf099
--- a/src/providers/ldap/sdap_dyndns.c
6cf099
+++ b/src/providers/ldap/sdap_dyndns.c
6cf099
@@ -112,10 +112,10 @@ sdap_dyndns_update_send(TALLOC_CTX *mem_ctx,
6cf099
     state->auth_type = auth_type;
6cf099
     state->pass_num = 0;
6cf099
 
6cf099
+    /* fallback servername is overriden by user option */
6cf099
     conf_servername = dp_opt_get_string(opts, DP_OPT_DYNDNS_SERVER);
6cf099
     if (conf_servername != NULL) {
6cf099
         state->servername = conf_servername;
6cf099
-        state->use_server_with_nsupdate = true;
6cf099
     }
6cf099
 
6cf099
     if (ifname) {
6cf099
-- 
6cf099
2.4.3
6cf099