[PATCH 2 of 2] cacert: improve error report when web.cacert file does not exist

timeless timeless at gmail.com
Sun Mar 6 09:57:24 CST 2011


# HG changeset patch
# User timeless <timeless at gmail.com>
# Date 1299426550 21600
# Node ID d27a39a67ad01faf4f45d869caa71cb4509473f7
# Parent  a2b1ee41dc4ef99c2297d7aab241b7141d1259c3
cacert: improve error report when web.cacert file does not exist

diff --git a/mercurial/url.py b/mercurial/url.py
--- a/mercurial/url.py
+++ b/mercurial/url.py
@@ -557,6 +557,10 @@ if has_https:
                 host = self.realhostport.rsplit(':', 1)[0]
 
             cacerts = self.ui.config('web', 'cacerts')
+            if cacerts and not os.path.exists(cacerts):
+                raise util.Abort(_('could not find '
+                                   'web.cacerts: %s\n') % cacerts)
+
             hostfingerprint = self.ui.config('hostfingerprints', host)
 
             if cacerts and not hostfingerprint:
diff --git a/tests/test-https.t b/tests/test-https.t
--- a/tests/test-https.t
+++ b/tests/test-https.t
@@ -96,6 +96,13 @@ pub.pem patched with other notBefore / n
   $ hg serve -p $HGPORT -d --pid-file=../hg0.pid --certificate=$PRIV
   $ cat ../hg0.pid >> $DAEMON_PIDS
 
+cacert not found
+
+  $ hg in --config web.cacerts=no-such.pem https://localhost:$HGPORT/
+  abort: could not find web.cacerts: no-such.pem
+  
+  [255]
+
 Test server address cannot be reused
 
   $ hg serve -p $HGPORT --certificate=$PRIV 2>&1


More information about the Mercurial-devel mailing list