From 9871f8d9ac9dc420e94d35209ebe0562cff92f69 Mon Sep 17 00:00:00 2001
From: Maxime Petazzoni <max@signalfuse.com>
Date: Thu, 7 Nov 2013 15:37:59 -0800
Subject: [PATCH] Don't fail loading the configuration when .dockercfg doesn't
 exist

Signed-off-by: Maxime Petazzoni <max@signalfuse.com>
---
 docker/auth/auth.py | 57 ++++++++++++++++++++++++---------------------
 tests/test.py       |  9 +++++++
 2 files changed, 39 insertions(+), 27 deletions(-)

diff --git a/docker/auth/auth.py b/docker/auth/auth.py
index c952f344..fc275590 100644
--- a/docker/auth/auth.py
+++ b/docker/auth/auth.py
@@ -98,33 +98,36 @@ def encode_header(auth):
 
 
 def load_config(root=None):
-    if root is None:
-        root = os.environ['HOME']
-    config_file = {
+    root = root or os.environ['HOME']
+    config = {
         'Configs': {},
         'rootPath': root
     }
-    f = open(os.path.join(root, '.dockercfg'))
-    try:
-        config_file['Configs'] = json.load(f)
-        for k, conf in six.iteritems(config_file['Configs']):
-            conf['Username'], conf['Password'] = decode_auth(conf['auth'])
-            del conf['auth']
-            config_file['Configs'][k] = conf
-    except Exception:
-        f.seek(0)
-        buf = []
-        for line in f:
-            k, v = line.split(' = ')
-            buf.append(v)
-        if len(buf) < 2:
-            raise Exception("The Auth config file is empty")
-        user, pwd = decode_auth(buf[0])
-        config_file['Configs'][INDEX_URL] = {
-            'Username': user,
-            'Password': pwd,
-            'Email': buf[1]
-        }
-    finally:
-        f.close()
-    return config_file
+
+    config_file = os.path.join(root, '.dockercfg')
+    if not os.path.exists(config_file):
+        return config
+
+    with open(config_file) as f:
+        try:
+            config['Configs'] = json.load(f)
+            for k, conf in six.iteritems(config['Configs']):
+                conf['Username'], conf['Password'] = decode_auth(conf['auth'])
+                del conf['auth']
+                config['Configs'][k] = conf
+        except Exception:
+            f.seek(0)
+            buf = []
+            for line in f:
+                k, v = line.split(' = ')
+                buf.append(v)
+            if len(buf) < 2:
+                raise Exception("The Auth config file is empty")
+            user, pwd = decode_auth(buf[0])
+            config['Configs'][INDEX_URL] = {
+                'Username': user,
+                'Password': pwd,
+                'Email': buf[1]
+            }
+
+    return config
diff --git a/tests/test.py b/tests/test.py
index f06d76eb..b7916f97 100644
--- a/tests/test.py
+++ b/tests/test.py
@@ -465,6 +465,15 @@ class DockerClientTest(unittest.TestCase):
     ## PY SPECIFIC TESTS ##
     #######################
 
+    def test_load_config_no_file(self):
+        folder = tempfile.mkdtemp()
+        cfg = docker.auth.load_config(folder)
+        self.assertTrue(cfg is not None)
+        self.assertIn('Configs', cfg)
+        self.assertEquals(cfg['Configs'], {})
+        self.assertIn('rootPath', cfg)
+        self.assertEquals(cfg['rootPath'], folder)
+
     def test_load_config(self):
         folder = tempfile.mkdtemp()
         f = open(os.path.join(folder, '.dockercfg'), 'w')