Merge pull request #2763 from migueldiascosta/fix_rest_headers

make REST client actually return headers when http method is HEAD
easybuilders · Feb 13, 2019 · 0862138 · 0862138
2 parents 8c760f2 + 983651b
commit 0862138
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/easybuild/base/rest.py b/easybuild/base/rest.py
@@ -169,11 +169,14 @@ def request(self, method, url, body, headers, content_type=None):
         # TODO: in recent python: Context manager
         conn = self.get_connection(method, url, body, headers)
         status = conn.code
-        body = conn.read()
-        try:
-            pybody = json.loads(body)
-        except ValueError:
-            pybody = body
+        if method == self.HEAD:
+            pybody = conn.headers
+        else:
+            body = conn.read()
+            try:
+                pybody = json.loads(body)
+            except ValueError:
+                pybody = body
         fancylogger.getLogger().debug('reponse len: %s ', len(pybody))
         conn.close()
         return status, pybody