[libvirt PATCH 08/10] remote: move proxy/mode defaults after URI parsing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Currently the defaults for the proxy/mode settings are set before
parsing URI parameters. A following commit will introduce a dependancy
on the URI parsing for the defaults, so they need to move.

Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
---
 src/remote/remote_driver.c | 50 ++++++++++++++++++++------------------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index 9ca4348f6f..6a881fff28 100644
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -757,20 +757,6 @@ doRemoteOpen(virConnectPtr conn,
     size_t i;
     int proxy;
 
-    if (inside_daemon && !conn->uri->server) {
-        mode = REMOTE_DRIVER_MODE_DIRECT;
-    } else {
-        mode = REMOTE_DRIVER_MODE_AUTO;
-    }
-
-    /* Historically we didn't allow ssh tunnel with session mode,
-     * since we can't construct the accurate path remotely,
-     * so we can default to modern virt-ssh-helper */
-    if (flags & REMOTE_DRIVER_OPEN_USER)
-        proxy = VIR_NET_CLIENT_PROXY_NATIVE;
-    else
-        proxy = VIR_NET_CLIENT_PROXY_AUTO;
-
     /* We handle *ALL* URIs here. The caller has rejected any
      * URIs we don't care about */
 
@@ -857,22 +843,38 @@ doRemoteOpen(virConnectPtr conn,
         virConfGetValueString(conf, "remote_mode", &mode_str) < 0)
         goto failed;
 
-    if (mode_str &&
-        (mode = remoteDriverModeTypeFromString(mode_str)) < 0) {
-        virReportError(VIR_ERR_INVALID_ARG,
-                       _("Unknown remote mode '%s'"), mode_str);
-        goto failed;
+    if (mode_str) {
+        if ((mode = remoteDriverModeTypeFromString(mode_str)) < 0) {
+            virReportError(VIR_ERR_INVALID_ARG,
+                           _("Unknown remote mode '%s'"), mode_str);
+            goto failed;
+        }
+    } else {
+        if (inside_daemon && !conn->uri->server) {
+            mode = REMOTE_DRIVER_MODE_DIRECT;
+        } else {
+            mode = REMOTE_DRIVER_MODE_AUTO;
+        }
     }
 
     if (conf && !proxy_str &&
         virConfGetValueString(conf, "remote_proxy", &proxy_str) < 0)
         goto failed;
 
-    if (proxy_str &&
-        (proxy = virNetClientProxyTypeFromString(proxy_str)) < 0) {
-        virReportError(VIR_ERR_INVALID_ARG,
-                       _("Unnkown proxy type '%s'"), proxy_str);
-        goto failed;
+    if (proxy_str) {
+        if ((proxy = virNetClientProxyTypeFromString(proxy_str)) < 0) {
+            virReportError(VIR_ERR_INVALID_ARG,
+                           _("Unnkown proxy type '%s'"), proxy_str);
+            goto failed;
+        }
+    } else {
+        /* Historically we didn't allow ssh tunnel with session mode,
+         * since we can't construct the accurate path remotely,
+         * so we can default to modern virt-ssh-helper */
+        if (flags & REMOTE_DRIVER_OPEN_USER)
+            proxy = VIR_NET_CLIENT_PROXY_NATIVE;
+        else
+            proxy = VIR_NET_CLIENT_PROXY_AUTO;
     }
 
     /* Sanity check that nothing requested !direct mode by mistake */
-- 
2.31.1




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux