stop expecting a JSON body for DELETE requests

Author: qrkourier

netfoundry/network.py

@@ -4,7 +4,7 @@
 import logging
 import re
 import time
-from requests.exceptions import HTTPError
+from requests.exceptions import HTTPError, JSONDecodeError
 
 from netfoundry.exceptions import UnknownResourceType, NetworkBoundaryViolation
 
@@ -1951,25 +1951,29 @@ def delete_resource(self, type: str, id: str = None, wait: int = 0, sleep: int =
         if response_code not in expected_responses:
             raise RuntimeError(f"got unexpected HTTP code {STATUS_CODES._codes[response_code][0].upper()} ({response_code}) and response {response.text}")
         else:
-            resource = response.json()
-
-        if resource.get('_links') and resource['_links'].get('process-executions'):
-            _links = resource['_links'].get('process-executions')
-            if isinstance(_links, list):
-                process_id = _links[0]['href'].split('/')[6]
-            else:
-                process_id = _links['href'].split('/')[6]
-            if wait:
-                self.wait_for_statuses(expected_statuses=RESOURCES["process-executions"].status_symbols['complete'], type="process-executions", id=process_id, wait=wait, sleep=sleep)
-                return(True)
-            else:    # only wait for the process to start, not finish, or timeout
-                self.wait_for_statuses(expected_statuses=RESOURCES['process-executions'].status_symbols['progress'] + RESOURCES['process-executions'].status_symbols['complete'], type="process-executions", id=process_id, wait=9, sleep=2)
+            try:
+                resource = response.json()
+            except JSONDecodeError as e:
+                logging.debug("ignoring {e}")
                 return(True)
-        elif wait:
-            logging.warning("unable to wait for async complete because response did not provide a process execution id")
-            return(False)
-        else:
-            return(True)
+            else:
+                if resource.get('_links') and resource['_links'].get('process-executions'):
+                    _links = resource['_links'].get('process-executions')
+                    if isinstance(_links, list):
+                        process_id = _links[0]['href'].split('/')[6]
+                    else:
+                        process_id = _links['href'].split('/')[6]
+                    if wait:
+                        self.wait_for_statuses(expected_statuses=RESOURCES["process-executions"].status_symbols['complete'], type="process-executions", id=process_id, wait=wait, sleep=sleep)
+                        return(True)
+                    else:    # only wait for the process to start, not finish, or timeout
+                        self.wait_for_statuses(expected_statuses=RESOURCES['process-executions'].status_symbols['progress'] + RESOURCES['process-executions'].status_symbols['complete'], type="process-executions", id=process_id, wait=9, sleep=2)
+                        return(True)
+                elif wait:
+                    logging.warning("unable to wait for async complete because response did not provide a process execution id")
+                    return(False)
+                else:
+                    return(True)
 
 
 class Networks: