Project

General

Profile

Actions
Copy link

Bug #7448

closed

XMLRPC Sync failure notice is ugly/long exception from cURL rather than our usual custom message

Added by Jim Pingle over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jim Pingle
Category:
XMLRPC
Target version:
2.4.0
Start date:
04/04/2017
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.4
Affected Architecture:
All

Description

On 2.4, if an XMLRPC sync attempt fails, a notice is displayed by the firewall but it contains a cURL exception error:

CurlException calling XMLRPC method host_firmware_version #HTTP_Request2_ConnectionException: Unable to connect to tls://172.16.1.3:443. Error: Operation timed out in /usr/local/share/pear/HTTP/Request2/Adapter/Socket.php on line 332
#0 /usr/local/share/pear/HTTP/Request2/Adapter/Socket.php(332): HTTP_Request2_SocketWrapper->__construct('tls://172.16.1....', 10, Array)
#1 /usr/local/share/pear/HTTP/Request2/Adapter/Socket.php(128): HTTP_Request2_Adapter_Socket->connect()
#2 /usr/local/share/pear/HTTP/Request2.php(946): HTTP_Request2_Adapter_Socket->sendRequest(Object(HTTP_Request2))
#3 /usr/local/share/pear/XML/RPC2/Util/HTTPRequest.php(234): HTTP_Request2->send()
#4 /usr/local/share/pear/XML/RPC2/Backend/Php/Client.php(115): XML_RPC2_Util_HTTPRequest->sendRequest()
#5 /etc/inc/xmlrpc_client.inc(102): XML_RPC2_Backend_Php_Client->__call('host_firmware_v...', Array)
#6 /etc/inc/xmlrpc_client.inc(102): XML_RPC2_Backend_Php_Client->host_firmware_version('')
#7 /etc/inc/xmlrpc_client.inc(152): pfsense_xmlrpc_client->xmlrpc_internal('host_firmware_v...', '', 240)
#8 /etc/rc.filter_synchronize(95): pfsense_xmlrpc_client->xmlrpc_method('host_firmware_v...')
#9 /etc/rc.filter_synchronize(356): carp_check_version()
#10 {main} @ 2017-04-04 13:41:08

In this past this was a much more friendly error message, such as "A communications error occurred while attempting XMLRPC sync ..." and it printed the username and target URL.

Looks like we need to catch that exception and setup our own handling and notice.

Actions
Copy link

Also available in: Atom PDF