Issue 639 prints in cdxj

ipwb/__main__.py

@@ -1,26 +1,23 @@
 import sys
-import os
 import argparse
 import tempfile
 import string  # For generating a temp file for stdin
 import random  # For generating a temp file for stdin
-from .__init__ import __version__ as ipwbVersion
+from .__init__ import __version__ as ipwb_version
 
 # ipwb modules
 from . import replay
 from . import indexer
-from . import util as ipwbUtil
-
-from .util import IPWBREPLAY_HOST, IPWBREPLAY_PORT
+from . import util
 
 
 def main():
-    ipwbUtil.checkForUpdate()
-    args = checkArgs(sys.argv)
+    util.check_pypi_for_update()
+    checkArgs(sys.argv)
 
 
 def checkArgs_index(args):
-    if not ipwbUtil.isDaemonAlive():
+    if not util.isDaemonAlive():
         sys.exit()
     encKey = None
     compressionLevel = None
@@ -140,11 +137,11 @@ def checkArgs(argsIn):
         '-d', '--daemon',
         help=("Multi-address of IPFS daemon "
               "(default /dns/localhost/tcp/5001/http)"),
-        default=ipwbUtil.IPFSAPI_MUTLIADDRESS,
+        default=util.IPFSAPI_MUTLIADDRESS,
         dest='daemon_address')
     parser.add_argument(
         '-v', '--version', help='Report the version of ipwb', action='version',
-        version='InterPlanetary Wayback ' + ipwbVersion)
+        version='InterPlanetary Wayback ' + ipwb_version)
 
     argCount = len(argsIn)
     cmdList = ['index', 'replay']

ipwb/util.py

@@ -1,7 +1,6 @@
 from __future__ import print_function
 
 from os.path import expanduser
-from os.path import basename
 
 import os
 import sys
@@ -15,17 +14,19 @@
 import logging
 import platform
 
-from six.moves.urllib.request import urlopen
 import json
-from .__init__ import __version__ as ipwbVersion
+from .__init__ import __version__ as ipwb_version
 
 from pkg_resources import parse_version
 
-# from requests.exceptions import ConnectionError
 from ipfshttpclient.exceptions import ConnectionError
 from ipfshttpclient.exceptions import AddressError
 from multiaddr.exceptions import StringParseError
 
+
+logger = logging.getLogger(__name__)
+
+
 IPFSAPI_MUTLIADDRESS = '/dns/localhost/tcp/5001/http'
 # or '/dns/{host}/tcp/{port}/http'
 # or '/ip4/{ipaddress}/tcp/{port}/http'
@@ -305,22 +306,28 @@ def unsurt(surt):
         return surt
 
 
-def compareCurrentAndLatestIPWBVersions():
+def compare_current_and_latest_ipwb_versions():
     try:
-        resp = urlopen('https://pypi.python.org/pypi/ipwb/json')
-        jResp = json.loads(resp.read())
-        latestVersion = jResp['info']['version']
-        currentVersion = re.sub(r'\.0+', '.', ipwbVersion)
-        return (currentVersion, latestVersion)
-    except Exception as e:
-        return (None, None)
+        response = requests.get('https://pypi.python.org/pypi/ipwb/json').json()
+
+        latest_version = response['info']['version']
+        current_version = re.sub(r'\.0+', '.', ipwb_version)
+
+        return current_version, latest_version
+
+    except Exception:
+        return None, None
 
 
-def checkForUpdate():
-    (current, latest) = compareCurrentAndLatestIPWBVersions()
+def check_pypi_for_update():
+    (current, latest) = compare_current_and_latest_ipwb_versions()
 
     if current != latest and current is not None:
-        print('This version of ipwb is outdated.'
-              ' Please run pip install --upgrade ipwb.')
-        print('* Latest version: {0}'.format(latest))
-        print('* Installed version: {0}'.format(current))
+        logger.warning(
+            'This version of ipwb is outdated. ' +
+            'Please run:\n\n' +
+            '   pip install --upgrade ipwb\n\n'
+            '* Latest version: %s\n' +
+            '* Installed version: %s',
+            latest, current
+        )