Change handle_httperror in httpapi plugins (#53391)

* Change return of handle_httperror

* Fix restconf for handle_httperror changes

lib/ansible/plugins/connection/httpapi.py

@@ -277,9 +277,9 @@ class Connection(NetworkConnectionBase):
             if is_handled is True:
                 return self.send(path, data, **kwargs)
             elif is_handled is False:
-                raise AnsibleConnectionFailure('Could not connect to {0}: {1}'.format(self._url + path, exc.reason))
-            else:
                 raise
+            else:
+                response = is_handled
         except URLError as exc:
             raise AnsibleConnectionFailure('Could not connect to {0}: {1}'.format(self._url + path, exc.reason))
 

lib/ansible/plugins/httpapi/__init__.py

@@ -61,11 +61,12 @@ class HttpApiBase(AnsiblePlugin):
         :returns:
             * True if the code has been handled in a way that the request
             may be resent without changes.
-            * False if this code indicates a fatal or unknown error which
-            cannot be handled by the plugin. This will result in an
-            AnsibleConnectionFailure being raised.
-            * Any other response passes the HTTPError along to the caller to
-            deal with as appropriate.
+            * False if the error cannot be handled or recovered from by the
+            plugin. This will result in the HTTPError being returned to the
+            caller to deal with as appropriate.
+            * Any other value returned is taken as a valid response from the
+            server without making another request. In many cases, this can just
+            be the original exception.
             """
         if exc.code == 401 and self.connection._auth:
             # Stored auth appears to be invalid, clear and retry
@@ -73,7 +74,7 @@ class HttpApiBase(AnsiblePlugin):
             self.login(self.connection.get_option('remote_user'), self.connection.get_option('password'))
             return True
 
-        return False
+        return exc
 
     @abstractmethod
     def send_request(self, data, **message_kwargs):

lib/ansible/plugins/httpapi/ftd.py

@@ -217,8 +217,8 @@ class HttpApi(HttpApiBase):
             self.connection._auth = None
             self.login(self.connection.get_option('remote_user'), self.connection.get_option('password'))
             return True
-        # None means that the exception will be passed further to the caller
-        return None
+        # False means that the exception will be passed further to the caller
+        return False
 
     def _display(self, http_method, title, msg=''):
         self.connection.queue_message('vvvv', 'REST:%s:%s:%s\n%s' % (http_method, self.connection._url, title, msg))

lib/ansible/plugins/httpapi/restconf.py

@@ -62,30 +62,26 @@ class HttpApi(HttpApiBase):
             'Content-Type': message_kwargs.get('content_type') or CONTENT_TYPE,
             'Accept': message_kwargs.get('accept') or CONTENT_TYPE,
         }
-        try:
         response, response_data = self.connection.send(path, data, headers=headers, method=message_kwargs.get('method'))
-        except HTTPError as exc:
-            response_data = exc
 
-        return handle_response(response_data)
-
-    def handle_httperror(self, exc):
-        return None
+        return handle_response(response, response_data)
 
 
-def handle_response(response):
+def handle_response(response, response_data):
     try:
-        response_json = json.loads(response.read())
+        response_data = json.loads(response_data.read())
     except ValueError:
+        response_data = response_data.read()
+
     if isinstance(response, HTTPError):
-            raise ConnectionError(to_text(response), code=response.code)
-        return response.read()
-
-    if 'errors' in response_json and 'jsonrpc' not in response_json:
-        errors = response_json['errors']['error']
-
+        if response_data:
+            if 'errors' in response_data:
+                errors = response_data['errors']['error']
                 error_text = '\n'.join((error['error-message'] for error in errors))
+            else:
+                error_text = response_data
 
             raise ConnectionError(error_text, code=response.code)
+        raise ConnectionError(to_text(response), code=response.code)
 
-    return response_json
+    return response_data