Refactor wo_download

wo/cli/plugins/update.py

@@ -50,6 +50,8 @@ class WOUpdateController(CementBaseController):
         if pargs.preserve:
             install_args = install_args + "--preserve "
 
+        if not os.path.isdir('/var/lib/wo/tmp'):
+            os.makedirs('/var/lib/wo/tmp')
         WODownload.download(self, [["https://raw.githubusercontent.com/"
                                     "WordOps/WordOps/{0}/install"
                                     .format(wo_branch),

wo/core/download.py

@@ -1,7 +1,6 @@
 """WordOps download core classes."""
 import os
-import urllib.error
+from requests import get, RequestException
-import urllib.request
 
 from wo.core.logging import Log
 
@@ -23,25 +22,15 @@ class WODownload():
                 if not os.path.exists(directory):
                     os.makedirs(directory)
                 Log.info(self, "Downloading {0:20}".format(pkg_name), end=' ')
-                req = urllib.request.Request(
+                with open(filename, "wb") as out_file:
-                    url, headers={'User-Agent': 'Mozilla/5.0'})
+                    req = get(url, timeout=15)
-                with urllib.request.urlopen(req) as response, open(filename, 'wb') as out_file:
+                    if req.encoding is None:
-                    out_file.write(response.read())
+                        req.encoding = 'utf-8'
+                    out_file.write(req.content)
                 Log.info(self, "{0}".format("[" + Log.ENDC + "Done"
                                             + Log.OKBLUE + "]"))
-            except urllib.error.URLError as e:
+            except RequestException as e:
                 Log.debug(self, "[{err}]".format(err=str(e.reason)))
                 Log.error(self, "Unable to download file, {0}"
                           .format(filename))
                 return False
-            except urllib.HTTPError.error as e:
-                Log.error(self, "Package download failed. {0}"
-                          .format(pkg_name))
-                Log.debug(self, "[{err}]".format(err=str(e.reason)))
-                return False
-            except urllib.ContentTooShortError.error as e:
-                Log.debug(self, "{0}{1}".format(e.errno, e.strerror))
-                Log.error(self, "Package download failed. The amount of the"
-                          " downloaded data is less than "
-                          "the expected amount \{0} ".format(pkg_name))
-                return False