mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 12:32:48 +01:00
bba215031b
14302a4802e2dbb41f5189de88f99ddd5dda7736 test: fix test abort for high timeout values (and `--timeout-factor 0`) (Sebastian Falbesoner) Pull request description: On master, the functional tests's option `--timeout-factor 0` (which according to the test docs and parameter description should disable the RPC timeouts) currently fails, same as high values like `--timeout-factor 999999`: ``` $ ./test/functional/wallet_basic.py --timeout-factor 0 2022-08-29T01:26:39.561000Z TestFramework (INFO): Initializing test directory /tmp/bitcoin_func_test_f24yxzp5 2022-08-29T01:26:40.262000Z TestFramework (ERROR): Assertion failed Traceback (most recent call last): File "/home/honey/bitcoin/test/functional/test_framework/test_framework.py", line 549, in start_nodes node.wait_for_rpc_connection() File "/home/honey/bitcoin/test/functional/test_framework/test_node.py", line 234, in wait_for_rpc_connection rpc.getblockcount() File "/home/honey/bitcoin/test/functional/test_framework/coverage.py", line 49, in __call__ return_val = self.auth_service_proxy_instance.__call__(*args, **kwargs) File "/home/honey/bitcoin/test/functional/test_framework/authproxy.py", line 142, in __call__ response, status = self._request('POST', self.__url.path, postdata.encode('utf-8')) File "/home/honey/bitcoin/test/functional/test_framework/authproxy.py", line 107, in _request self.__conn.request(method, path, postdata, headers) File "/usr/local/lib/python3.9/http/client.py", line 1285, in request self._send_request(method, url, body, headers, encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1331, in _send_request self.endheaders(body, encode_chunked=encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1280, in endheaders self._send_output(message_body, encode_chunked=encode_chunked) File "/usr/local/lib/python3.9/http/client.py", line 1040, in _send_output self.send(msg) File "/usr/local/lib/python3.9/http/client.py", line 980, in send self.connect() File "/usr/local/lib/python3.9/http/client.py", line 946, in connect self.sock = self._create_connection( File "/usr/local/lib/python3.9/socket.py", line 844, in create_connection raise err File "/usr/local/lib/python3.9/socket.py", line 832, in create_connection sock.connect(sa) OSError: [Errno 22] Invalid argument ``` This is caused by a high timeout value that Python's HTTP(S) client library can't cope with. Fix this by clamping down the connection's set timeout value in AuthProxy. The change can easily be tested by running an arbitrary test with `--timeout-factor 0` on master (should fail), on this PR (should pass) and on this PR with the clamping value increased by 1 (should fail). // EDIT: The behaviour was observed on OpenBSD 7.1 and Python 3.9.12. ACKs for top commit: MarcoFalke: lgtm ACK 14302a4802e2dbb41f5189de88f99ddd5dda7736 Tree-SHA512: 6469e8ac699f1bb7dea11d5fb8b3ae54d895bb908570587c5631144cd41fe980ca0b1e6d0b7bfa07983307cba15fb26ae92e6766375672bf5be838d8e5422dbc
212 lines
9.0 KiB
Python
212 lines
9.0 KiB
Python
# Copyright (c) 2011 Jeff Garzik
|
|
#
|
|
# Previous copyright, from python-jsonrpc/jsonrpc/proxy.py:
|
|
#
|
|
# Copyright (c) 2007 Jan-Klaas Kollhof
|
|
#
|
|
# This file is part of jsonrpc.
|
|
#
|
|
# jsonrpc is free software; you can redistribute it and/or modify
|
|
# it under the terms of the GNU Lesser General Public License as published by
|
|
# the Free Software Foundation; either version 2.1 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# This software is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU Lesser General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU Lesser General Public License
|
|
# along with this software; if not, write to the Free Software
|
|
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
|
|
"""HTTP proxy for opening RPC connection to dashd.
|
|
|
|
AuthServiceProxy has the following improvements over python-jsonrpc's
|
|
ServiceProxy class:
|
|
|
|
- HTTP connections persist for the life of the AuthServiceProxy object
|
|
(if server supports HTTP/1.1)
|
|
- sends protocol 'version', per JSON-RPC 1.1
|
|
- sends proper, incrementing 'id'
|
|
- sends Basic HTTP authentication headers
|
|
- parses all JSON numbers that look like floats as Decimal
|
|
- uses standard Python json lib
|
|
"""
|
|
|
|
import base64
|
|
import decimal
|
|
from http import HTTPStatus
|
|
import http.client
|
|
import json
|
|
import logging
|
|
import os
|
|
import socket
|
|
import time
|
|
import urllib.parse
|
|
|
|
HTTP_TIMEOUT = 30
|
|
USER_AGENT = "AuthServiceProxy/0.1"
|
|
|
|
log = logging.getLogger("BitcoinRPC")
|
|
|
|
class JSONRPCException(Exception):
|
|
def __init__(self, rpc_error, http_status=None):
|
|
try:
|
|
errmsg = '%(message)s (%(code)i)' % rpc_error
|
|
except (KeyError, TypeError):
|
|
errmsg = ''
|
|
super().__init__(errmsg)
|
|
self.error = rpc_error
|
|
self.http_status = http_status
|
|
|
|
|
|
def EncodeDecimal(o):
|
|
if isinstance(o, decimal.Decimal):
|
|
return str(o)
|
|
raise TypeError(repr(o) + " is not JSON serializable")
|
|
|
|
class AuthServiceProxy():
|
|
__id_count = 0
|
|
|
|
# ensure_ascii: escape unicode as \uXXXX, passed to json.dumps
|
|
def __init__(self, service_url, service_name=None, timeout=HTTP_TIMEOUT, connection=None, ensure_ascii=True):
|
|
self.__service_url = service_url
|
|
self._service_name = service_name
|
|
self.ensure_ascii = ensure_ascii # can be toggled on the fly by tests
|
|
self.__url = urllib.parse.urlparse(service_url)
|
|
user = None if self.__url.username is None else self.__url.username.encode('utf8')
|
|
passwd = None if self.__url.password is None else self.__url.password.encode('utf8')
|
|
authpair = user + b':' + passwd
|
|
self.__auth_header = b'Basic ' + base64.b64encode(authpair)
|
|
# clamp the socket timeout, since larger values can cause an
|
|
# "Invalid argument" exception in Python's HTTP(S) client
|
|
# library on some operating systems (e.g. OpenBSD, FreeBSD)
|
|
self.timeout = min(timeout, 2147483)
|
|
self._set_conn(connection)
|
|
|
|
def __getattr__(self, name):
|
|
if name.startswith('__') and name.endswith('__'):
|
|
# Python internal stuff
|
|
raise AttributeError
|
|
if self._service_name is not None:
|
|
name = "%s.%s" % (self._service_name, name)
|
|
return AuthServiceProxy(self.__service_url, name, connection=self.__conn)
|
|
|
|
def _request(self, method, path, postdata):
|
|
'''
|
|
Do a HTTP request, with retry if we get disconnected (e.g. due to a timeout).
|
|
This is a workaround for https://bugs.python.org/issue3566 which is fixed in Python 3.5.
|
|
'''
|
|
headers = {'Host': self.__url.hostname,
|
|
'User-Agent': USER_AGENT,
|
|
'Authorization': self.__auth_header,
|
|
'Content-type': 'application/json'}
|
|
if os.name == 'nt':
|
|
# Windows somehow does not like to re-use connections
|
|
# TODO: Find out why the connection would disconnect occasionally and make it reusable on Windows
|
|
# Avoid "ConnectionAbortedError: [WinError 10053] An established connection was aborted by the software in your host machine"
|
|
self._set_conn()
|
|
try:
|
|
self.__conn.request(method, path, postdata, headers)
|
|
return self._get_response()
|
|
except (BrokenPipeError, ConnectionResetError):
|
|
# Python 3.5+ raises BrokenPipeError when the connection was reset
|
|
# ConnectionResetError happens on FreeBSD
|
|
self.__conn.close()
|
|
self.__conn.request(method, path, postdata, headers)
|
|
return self._get_response()
|
|
except OSError as e:
|
|
retry = (
|
|
'[WinError 10053] An established connection was aborted by the software in your host machine' in str(e))
|
|
# Workaround for a bug on macOS. See https://bugs.python.org/issue33450
|
|
retry = retry or ('[Errno 41] Protocol wrong type for socket' in str(e))
|
|
if retry:
|
|
self.__conn.close()
|
|
self.__conn.request(method, path, postdata, headers)
|
|
return self._get_response()
|
|
else:
|
|
raise
|
|
|
|
def get_request(self, *args, **argsn):
|
|
AuthServiceProxy.__id_count += 1
|
|
|
|
log.debug("-{}-> {} {}".format(
|
|
AuthServiceProxy.__id_count,
|
|
self._service_name,
|
|
json.dumps(args or argsn, default=EncodeDecimal, ensure_ascii=self.ensure_ascii),
|
|
))
|
|
if args and argsn:
|
|
params = dict(args=args, **argsn)
|
|
else:
|
|
params = args or argsn
|
|
return {'version': '1.1',
|
|
'method': self._service_name,
|
|
'params': params,
|
|
'id': AuthServiceProxy.__id_count}
|
|
|
|
def __call__(self, *args, **argsn):
|
|
postdata = json.dumps(self.get_request(*args, **argsn), default=EncodeDecimal, ensure_ascii=self.ensure_ascii)
|
|
response, status = self._request('POST', self.__url.path, postdata.encode('utf-8'))
|
|
if response['error'] is not None:
|
|
raise JSONRPCException(response['error'], status)
|
|
elif 'result' not in response:
|
|
raise JSONRPCException({
|
|
'code': -343, 'message': 'missing JSON-RPC result'}, status)
|
|
elif status != HTTPStatus.OK:
|
|
raise JSONRPCException({
|
|
'code': -342, 'message': 'non-200 HTTP status code but no JSON-RPC error'}, status)
|
|
else:
|
|
return response['result']
|
|
|
|
def batch(self, rpc_call_list):
|
|
postdata = json.dumps(list(rpc_call_list), default=EncodeDecimal, ensure_ascii=self.ensure_ascii)
|
|
log.debug("--> " + postdata)
|
|
response, status = self._request('POST', self.__url.path, postdata.encode('utf-8'))
|
|
if status != HTTPStatus.OK:
|
|
raise JSONRPCException({
|
|
'code': -342, 'message': 'non-200 HTTP status code but no JSON-RPC error'}, status)
|
|
return response
|
|
|
|
def _get_response(self):
|
|
req_start_time = time.time()
|
|
try:
|
|
http_response = self.__conn.getresponse()
|
|
except socket.timeout:
|
|
raise JSONRPCException({
|
|
'code': -344,
|
|
'message': '%r RPC took longer than %f seconds. Consider '
|
|
'using larger timeout for calls that take '
|
|
'longer to return.' % (self._service_name,
|
|
self.__conn.timeout)})
|
|
if http_response is None:
|
|
raise JSONRPCException({
|
|
'code': -342, 'message': 'missing HTTP response from server'})
|
|
|
|
content_type = http_response.getheader('Content-Type')
|
|
if content_type != 'application/json':
|
|
raise JSONRPCException(
|
|
{'code': -342, 'message': 'non-JSON HTTP response with \'%i %s\' from server' % (http_response.status, http_response.reason)},
|
|
http_response.status)
|
|
|
|
responsedata = http_response.read().decode('utf8')
|
|
response = json.loads(responsedata, parse_float=decimal.Decimal)
|
|
elapsed = time.time() - req_start_time
|
|
if "error" in response and response["error"] is None:
|
|
log.debug("<-%s- [%.6f] %s" % (response["id"], elapsed, json.dumps(response["result"], default=EncodeDecimal, ensure_ascii=self.ensure_ascii)))
|
|
else:
|
|
log.debug("<-- [%.6f] %s" % (elapsed, responsedata))
|
|
return response, http_response.status
|
|
|
|
def __truediv__(self, relative_uri):
|
|
return AuthServiceProxy("{}/{}".format(self.__service_url, relative_uri), self._service_name, connection=self.__conn)
|
|
|
|
def _set_conn(self, connection=None):
|
|
port = 80 if self.__url.port is None else self.__url.port
|
|
if connection:
|
|
self.__conn = connection
|
|
self.timeout = connection.timeout
|
|
elif self.__url.scheme == 'https':
|
|
self.__conn = http.client.HTTPSConnection(self.__url.hostname, port, timeout=self.timeout)
|
|
else:
|
|
self.__conn = http.client.HTTPConnection(self.__url.hostname, port, timeout=self.timeout)
|