mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 12:32:48 +01:00
186a517692
This avoids a regression for issues like #334 where high speed repeated connections eventually run the HTTP client out of sockets because all of theirs end up in time_wait. Maybe the trade-off here is suboptimal, but if both choices will fail then we prefer fewer changes until the root cause is solved. Rebased-From:1a25a7edf87d2cb48511Github-Pull: #5674
103 lines
4.6 KiB
Python
Executable File
103 lines
4.6 KiB
Python
Executable File
#!/usr/bin/env python2
|
|
# Copyright (c) 2014 The Bitcoin Core developers
|
|
# Distributed under the MIT software license, see the accompanying
|
|
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
|
|
|
|
#
|
|
# Test REST interface
|
|
#
|
|
|
|
from test_framework import BitcoinTestFramework
|
|
from util import *
|
|
import base64
|
|
|
|
try:
|
|
import http.client as httplib
|
|
except ImportError:
|
|
import httplib
|
|
try:
|
|
import urllib.parse as urlparse
|
|
except ImportError:
|
|
import urlparse
|
|
|
|
class HTTPBasicsTest (BitcoinTestFramework):
|
|
def setup_nodes(self):
|
|
return start_nodes(4, self.options.tmpdir, extra_args=[['-rpckeepalive=1'], ['-rpckeepalive=0'], [], []])
|
|
|
|
def run_test(self):
|
|
|
|
#################################################
|
|
# lowlevel check for http persistent connection #
|
|
#################################################
|
|
url = urlparse.urlparse(self.nodes[0].url)
|
|
authpair = url.username + ':' + url.password
|
|
headers = {"Authorization": "Basic " + base64.b64encode(authpair)}
|
|
|
|
conn = httplib.HTTPConnection(url.hostname, url.port)
|
|
conn.connect()
|
|
conn.request('POST', '/', '{"method": "getbestblockhash"}', headers)
|
|
out1 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True)
|
|
assert_equal(conn.sock!=None, True) #according to http/1.1 connection must still be open!
|
|
|
|
#send 2nd request without closing connection
|
|
conn.request('POST', '/', '{"method": "getchaintips"}', headers)
|
|
out2 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True) #must also response with a correct json-rpc message
|
|
assert_equal(conn.sock!=None, True) #according to http/1.1 connection must still be open!
|
|
conn.close()
|
|
|
|
#same should be if we add keep-alive because this should be the std. behaviour
|
|
headers = {"Authorization": "Basic " + base64.b64encode(authpair), "Connection": "keep-alive"}
|
|
|
|
conn = httplib.HTTPConnection(url.hostname, url.port)
|
|
conn.connect()
|
|
conn.request('POST', '/', '{"method": "getbestblockhash"}', headers)
|
|
out1 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True)
|
|
assert_equal(conn.sock!=None, True) #according to http/1.1 connection must still be open!
|
|
|
|
#send 2nd request without closing connection
|
|
conn.request('POST', '/', '{"method": "getchaintips"}', headers)
|
|
out2 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True) #must also response with a correct json-rpc message
|
|
assert_equal(conn.sock!=None, True) #according to http/1.1 connection must still be open!
|
|
conn.close()
|
|
|
|
#now do the same with "Connection: close"
|
|
headers = {"Authorization": "Basic " + base64.b64encode(authpair), "Connection":"close"}
|
|
|
|
conn = httplib.HTTPConnection(url.hostname, url.port)
|
|
conn.connect()
|
|
conn.request('POST', '/', '{"method": "getbestblockhash"}', headers)
|
|
out1 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True)
|
|
assert_equal(conn.sock!=None, False) #now the connection must be closed after the response
|
|
|
|
#node1 (2nd node) is running with disabled keep-alive option
|
|
urlNode1 = urlparse.urlparse(self.nodes[1].url)
|
|
authpair = urlNode1.username + ':' + urlNode1.password
|
|
headers = {"Authorization": "Basic " + base64.b64encode(authpair)}
|
|
|
|
conn = httplib.HTTPConnection(urlNode1.hostname, urlNode1.port)
|
|
conn.connect()
|
|
conn.request('POST', '/', '{"method": "getbestblockhash"}', headers)
|
|
out1 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True)
|
|
assert_equal(conn.sock!=None, False) #connection must be closed because keep-alive was set to false
|
|
|
|
#node2 (third node) is running with standard keep-alive parameters which means keep-alive is off
|
|
urlNode2 = urlparse.urlparse(self.nodes[2].url)
|
|
authpair = urlNode2.username + ':' + urlNode2.password
|
|
headers = {"Authorization": "Basic " + base64.b64encode(authpair)}
|
|
|
|
conn = httplib.HTTPConnection(urlNode2.hostname, urlNode2.port)
|
|
conn.connect()
|
|
conn.request('POST', '/', '{"method": "getbestblockhash"}', headers)
|
|
out1 = conn.getresponse().read();
|
|
assert_equal('"error":null' in out1, True)
|
|
assert_equal(conn.sock!=None, True) #connection must be closed because bitcoind should use keep-alive by default
|
|
|
|
if __name__ == '__main__':
|
|
HTTPBasicsTest ().main ()
|