py3: use node.hex(h.digest()) instead of h.hexdigest()
authorPulkit Goyal <7895pulkit@gmail.com>
Fri, 29 Dec 2017 05:25:27 +0530
changeset 35582 72b91f905065
parent 35581 154754d1f137
child 35583 2f123f309f61
py3: use node.hex(h.digest()) instead of h.hexdigest() hashlib.sha1.hexdigest() returns str on Python 3. Differential Revision: https://phab.mercurial-scm.org/D1792
mercurial/chgserver.py
mercurial/hg.py
mercurial/keepalive.py
mercurial/patch.py
mercurial/sparse.py
mercurial/sslutil.py
mercurial/store.py
mercurial/subrepo.py
mercurial/util.py
--- a/mercurial/chgserver.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/chgserver.py	Fri Dec 29 05:25:27 2017 +0530
@@ -55,6 +55,7 @@
     encoding,
     error,
     extensions,
+    node,
     pycompat,
     util,
 )
@@ -63,7 +64,7 @@
 
 def _hashlist(items):
     """return sha1 hexdigest for a list"""
-    return hashlib.sha1(str(items)).hexdigest()
+    return node.hex(hashlib.sha1(str(items)).digest())
 
 # sensitive config sections affecting confighash
 _configsections = [
--- a/mercurial/hg.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/hg.py	Fri Dec 29 05:25:27 2017 +0530
@@ -569,7 +569,7 @@
                             'unable to resolve identity of remote)\n'))
         elif sharenamemode == 'remote':
             sharepath = os.path.join(
-                sharepool, hashlib.sha1(source).hexdigest())
+                sharepool, node.hex(hashlib.sha1(source).digest()))
         else:
             raise error.Abort(_('unknown share naming mode: %s') %
                               sharenamemode)
--- a/mercurial/keepalive.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/keepalive.py	Fri Dec 29 05:25:27 2017 +0530
@@ -92,6 +92,7 @@
 
 from .i18n import _
 from . import (
+    node,
     pycompat,
     urllibcompat,
     util,
@@ -607,7 +608,7 @@
     foo = fo.read()
     fo.close()
     m = md5(foo)
-    print(format % ('normal urllib', m.hexdigest()))
+    print(format % ('normal urllib', node.hex(m.digest())))
 
     # now install the keepalive handler and try again
     opener = urlreq.buildopener(HTTPHandler())
@@ -617,7 +618,7 @@
     foo = fo.read()
     fo.close()
     m = md5(foo)
-    print(format % ('keepalive read', m.hexdigest()))
+    print(format % ('keepalive read', node.hex(m.digest())))
 
     fo = urlreq.urlopen(url)
     foo = ''
@@ -629,7 +630,7 @@
             break
     fo.close()
     m = md5(foo)
-    print(format % ('keepalive readline', m.hexdigest()))
+    print(format % ('keepalive readline', node.hex(m.digest())))
 
 def comp(N, url):
     print('  making %i connections to:\n  %s' % (N, url))
--- a/mercurial/patch.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/patch.py	Fri Dec 29 05:25:27 2017 +0530
@@ -2654,7 +2654,7 @@
         l = len(text)
         s = hashlib.sha1('blob %d\0' % l)
         s.update(text)
-        return s.hexdigest()
+        return hex(s.digest())
 
     if opts.noprefix:
         aprefix = bprefix = ''
--- a/mercurial/sparse.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/sparse.py	Fri Dec 29 05:25:27 2017 +0530
@@ -12,7 +12,10 @@
 import os
 
 from .i18n import _
-from .node import nullid
+from .node import (
+    hex,
+    nullid,
+)
 from . import (
     error,
     match as matchmod,
@@ -173,12 +176,12 @@
         tempsignature = '0'
 
     if signature is None or (includetemp and tempsignature is None):
-        signature = hashlib.sha1(repo.vfs.tryread('sparse')).hexdigest()
+        signature = hex(hashlib.sha1(repo.vfs.tryread('sparse')).digest())
         cache['signature'] = signature
 
         if includetemp:
             raw = repo.vfs.tryread('tempsparse')
-            tempsignature = hashlib.sha1(raw).hexdigest()
+            tempsignature = hex(hashlib.sha1(raw).digest())
             cache['tempsignature'] = tempsignature
 
     return '%s %s' % (signature, tempsignature)
--- a/mercurial/sslutil.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/sslutil.py	Fri Dec 29 05:25:27 2017 +0530
@@ -17,6 +17,7 @@
 from .i18n import _
 from . import (
     error,
+    node,
     pycompat,
     util,
 )
@@ -808,9 +809,9 @@
     # If a certificate fingerprint is pinned, use it and only it to
     # validate the remote cert.
     peerfingerprints = {
-        'sha1': hashlib.sha1(peercert).hexdigest(),
-        'sha256': hashlib.sha256(peercert).hexdigest(),
-        'sha512': hashlib.sha512(peercert).hexdigest(),
+        'sha1': node.hex(hashlib.sha1(peercert).digest()),
+        'sha256': node.hex(hashlib.sha256(peercert).digest()),
+        'sha512': node.hex(hashlib.sha512(peercert).digest()),
     }
 
     def fmtfingerprint(s):
--- a/mercurial/store.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/store.py	Fri Dec 29 05:25:27 2017 +0530
@@ -15,6 +15,7 @@
 from .i18n import _
 from . import (
     error,
+    node,
     policy,
     pycompat,
     util,
@@ -221,7 +222,7 @@
 _maxshortdirslen = 8 * (_dirprefixlen + 1) - 4
 
 def _hashencode(path, dotencode):
-    digest = hashlib.sha1(path).hexdigest()
+    digest = node.hex(hashlib.sha1(path).digest())
     le = lowerencode(path[5:]).split('/') # skips prefix 'data/' or 'meta/'
     parts = _auxencode(le, dotencode)
     basename = parts[-1]
--- a/mercurial/subrepo.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/subrepo.py	Fri Dec 29 05:25:27 2017 +0530
@@ -55,7 +55,7 @@
 
 def _getstorehashcachename(remotepath):
     '''get a unique filename for the store hash cache of a remote repository'''
-    return hashlib.sha1(_expandedabspath(remotepath)).hexdigest()[0:12]
+    return node.hex(hashlib.sha1(_expandedabspath(remotepath)).digest())[0:12]
 
 class SubrepoAbort(error.Abort):
     """Exception class used to avoid handling a subrepo error more than once"""
@@ -789,7 +789,7 @@
         yield '# %s\n' % _expandedabspath(remotepath)
         vfs = self._repo.vfs
         for relname in filelist:
-            filehash = hashlib.sha1(vfs.tryread(relname)).hexdigest()
+            filehash = node.hex(hashlib.sha1(vfs.tryread(relname)).digest())
             yield '%s = %s\n' % (relname, filehash)
 
     @propertycache
--- a/mercurial/util.py	Fri Dec 29 05:22:06 2017 +0530
+++ b/mercurial/util.py	Fri Dec 29 05:25:27 2017 +0530
@@ -49,6 +49,7 @@
     encoding,
     error,
     i18n,
+    node as nodemod,
     policy,
     pycompat,
     urllibcompat,
@@ -265,7 +266,7 @@
     def __getitem__(self, key):
         if key not in DIGESTS:
             raise Abort(_('unknown digest type: %s') % k)
-        return self._hashes[key].hexdigest()
+        return nodemod.hex(self._hashes[key].digest())
 
     def __iter__(self):
         return iter(self._hashes)