centralize getting the hash because duh

_base.py

@@ -90,17 +90,16 @@ class BaseUpdater(object):
             return(None)
         if not self.iso_url:
             raise RuntimeError('iso_url attribute must be set first')
-        req = requests.get(self.iso_url, stream = True, headers = {'User-Agent': 'curl/7.74.0'})
+        req_chk = requests.head(self.iso_url, headers = {'User-Agent': 'curl/7.74.0'})
-        if not req.ok:
+        if not req_chk.ok:
-            raise RuntimeError('Received non-200/30x {0} for {1}'.format(req.status_code, self.iso_url))
+            raise RuntimeError('Received non-200/30x {0} for {1}'.format(req_chk.status_code, self.iso_url))
         os.makedirs(os.path.dirname(self.dest_iso), exist_ok = True)
-        with req as uri:
+        with requests.get(self.iso_url, stream = True, headers = {'User-Agent': 'curl/7.74.0'}) as req:
+            req.raise_for_status()
             with open(self.dest_iso, 'wb') as fh:
-                shutil.copyfileobj(uri.raw, fh)
+                for chunk in req.iter_content(chunk_size = 8192):
-        hasher = hashlib.new(self.hash_type)
+                    fh.write(chunk)
-        with open(self.dest_iso, 'rb') as fh:
+        realhash = self.getISOHash()
-            hasher.update(fh.read())
-        realhash = hasher.hexdigest().lower()
         if realhash != self.new_hash:
             raise RuntimeError('Hash mismatch: {0} (LOCAL), {1} (REMOTE)'.format(realhash, self.new_hash))
         self.updateVer()
@@ -110,6 +109,17 @@ class BaseUpdater(object):
         raise RuntimeError('BaseUpdater should be subclassed and its updateVer, getCurVer, and getNewVer methods '
                            'should be replaced.')
 
+    def getISOHash(self):
+        hasher = hashlib.new(self.hash_type)
+        # TODO: later on when python 3.8 is more prevalent, https://stackoverflow.com/a/1131238/733214
+        with open(self.dest_iso, 'rb') as fh:
+            while True:
+                chunk = fh.read(8192)
+                if not chunk:
+                    break
+                hasher.update(chunk)
+        return(hasher.hexdigest().lower())
+
     def getNewVer(self):
         raise RuntimeError('BaseUpdater should be subclassed and its updateVer, getCurVer, and getNewVer methods '
                            'should be replaced.')

arch.py

@@ -13,7 +13,6 @@
 
 import datetime
 import json
-import hashlib
 import os
 import re
 ##
@@ -133,17 +132,14 @@ class Updater(_base.BaseUpdater):
         # if ver_info.get('arch') != self.arch:
         #     self.do_update = True
         #     self.force_update = True
-        try:
+        if not os.path.isfile(self.dest_iso):
-            hasher = hashlib.new(self.hash_type)
-            with open(self.dest_iso, 'rb') as fh:
-                hasher.update(fh.read())
-            if self.old_hash != hasher.hexdigest().lower():
-                self.do_update = True
-                self.force_update = True
-        except FileNotFoundError:
             self.do_update = True
             self.force_update = True
             return(None)
+        realhash = self.getISOHash()
+        if self.old_hash != realhash:
+            self.do_update = True
+            self.force_update = True
         return(None)
 
     def getNet(self):

grml.py

@@ -10,7 +10,6 @@
 
 import datetime
 import json
-import hashlib
 import os
 import re
 ##
@@ -94,17 +93,14 @@ class Updater(_base.BaseUpdater):
         if ver_info.get('arch') != self.arch:
             self.do_update = True
             self.force_update = True
-        try:
+        if not os.path.isfile(self.dest_iso):
-            hasher = hashlib.new(self.hash_type)
-            with open(self.dest_iso, 'rb') as fh:
-                hasher.update(fh.read())
-            if self.old_hash != hasher.hexdigest().lower():
-                self.do_update = True
-                self.force_update = True
-        except FileNotFoundError:
             self.do_update = True
             self.force_update = True
             return(None)
+        realhash = self.getISOHash()
+        if self.old_hash != realhash:
+            self.do_update = True
+            self.force_update = True
         return(None)
 
     def getNewVer(self):

sysresccd.py

@@ -10,7 +10,6 @@
 
 import datetime
 import json
-import hashlib
 import os
 import re
 ##
@@ -78,17 +77,14 @@ class Updater(_base.BaseUpdater):
         if ver_info.get('arch') != self.arch:
             self.do_update = True
             self.force_update = True
-        try:
+        if not os.path.isfile(self.dest_iso):
-            hasher = hashlib.new(self.hash_type)
+            self.do_update = True
-            with open(self.dest_iso, 'rb') as fh:
+            self.force_update = True
-                hasher.update(fh.read())
+            return (None)
-            if self.old_hash != hasher.hexdigest().lower():
+        realhash = self.getISOHash()
-                self.do_update = True
+        if self.old_hash != realhash:
-                self.force_update = True
-        except FileNotFoundError:
             self.do_update = True
             self.force_update = True
-            return(None)
         return (None)
 
     def getNewVer(self):