Correctly load configuration when instantiating new client without kwargs

The init function of PodmanClient was never loading the system configuration
when no kwargs were specified. This change makes it so that we load the core
data (base_url and identity) from the config file.

Signed-off-by: Anish Asthana <[email protected]>

Author: anishasthana

podman/client.py

@@ -62,15 +62,21 @@ def __init__(self, **kwargs) -> None:
 
         api_kwargs = kwargs.copy()
 
-        if "connection" in api_kwargs:
-            connection = config.services[api_kwargs.get("connection")]
-            api_kwargs["base_url"] = connection.url.geturl()
-
-            # Override configured identity, if provided in arguments
-            api_kwargs["identity"] = kwargs.get("identity", str(connection.identity))
-        elif "base_url" not in api_kwargs:
-            path = str(Path(get_runtime_dir()) / "podman" / "podman.sock")
-            api_kwargs["base_url"] = "http+unix://" + path
+        if not api_kwargs:
+            if "Connection" in config.attrs:
+                default_connection_name = config.attrs["Connection"]["Default"]
+                connection = config.attrs["Connection"]["Connections"][default_connection_name]
+                api_kwargs["base_url"] = connection["URI"]
+                api_kwargs["identity"] = connection["Identity"]
+        else:
+            if "connection" in api_kwargs:
+                connection = config.services[api_kwargs.get("connection")]
+                api_kwargs["base_url"] = connection.url.geturl()
+                # Override configured identity, if provided in arguments
+                api_kwargs["identity"] = kwargs.get("identity", str(connection.identity))
+            elif "base_url" not in api_kwargs:
+                path = str(Path(get_runtime_dir()) / "podman" / "podman.sock")
+                api_kwargs["base_url"] = "http+unix://" + path
         self.api = APIClient(**api_kwargs)
 
     def __enter__(self) -> "PodmanClient":