Stop passing even a dummy value for control tub into Nodes

This commit is contained in:
Jean-Paul Calderone 2021-10-15 13:10:18 -04:00
parent fe2e2cc1d6
commit 0611af6b0b
5 changed files with 6 additions and 11 deletions

View File

@ -282,7 +282,6 @@ def create_client_from_config(config, _client_factory=None, _introducer_factory=
config, tub_options, default_connection_handlers, config, tub_options, default_connection_handlers,
foolscap_connection_handlers, i2p_provider, tor_provider, foolscap_connection_handlers, i2p_provider, tor_provider,
) )
control_tub = None
introducer_clients = create_introducer_clients(config, main_tub, _introducer_factory) introducer_clients = create_introducer_clients(config, main_tub, _introducer_factory)
storage_broker = create_storage_farm_broker( storage_broker = create_storage_farm_broker(
@ -293,7 +292,6 @@ def create_client_from_config(config, _client_factory=None, _introducer_factory=
client = _client_factory( client = _client_factory(
config, config,
main_tub, main_tub,
control_tub,
i2p_provider, i2p_provider,
tor_provider, tor_provider,
introducer_clients, introducer_clients,
@ -630,12 +628,12 @@ class _Client(node.Node, pollmixin.PollMixin):
"max_segment_size": DEFAULT_MAX_SEGMENT_SIZE, "max_segment_size": DEFAULT_MAX_SEGMENT_SIZE,
} }
def __init__(self, config, main_tub, control_tub, i2p_provider, tor_provider, introducer_clients, def __init__(self, config, main_tub, i2p_provider, tor_provider, introducer_clients,
storage_farm_broker): storage_farm_broker):
""" """
Use :func:`allmydata.client.create_client` to instantiate one of these. Use :func:`allmydata.client.create_client` to instantiate one of these.
""" """
node.Node.__init__(self, config, main_tub, control_tub, i2p_provider, tor_provider) node.Node.__init__(self, config, main_tub, i2p_provider, tor_provider)
self.started_timestamp = time.time() self.started_timestamp = time.time()
self.logSource = "Client" self.logSource = "Client"

View File

@ -87,12 +87,10 @@ def create_introducer(basedir=u"."):
config, tub_options, default_connection_handlers, config, tub_options, default_connection_handlers,
foolscap_connection_handlers, i2p_provider, tor_provider, foolscap_connection_handlers, i2p_provider, tor_provider,
) )
control_tub = None
node = _IntroducerNode( node = _IntroducerNode(
config, config,
main_tub, main_tub,
control_tub,
i2p_provider, i2p_provider,
tor_provider, tor_provider,
) )
@ -104,8 +102,8 @@ def create_introducer(basedir=u"."):
class _IntroducerNode(node.Node): class _IntroducerNode(node.Node):
NODETYPE = "introducer" NODETYPE = "introducer"
def __init__(self, config, main_tub, control_tub, i2p_provider, tor_provider): def __init__(self, config, main_tub, i2p_provider, tor_provider):
node.Node.__init__(self, config, main_tub, control_tub, i2p_provider, tor_provider) node.Node.__init__(self, config, main_tub, i2p_provider, tor_provider)
self.init_introducer() self.init_introducer()
webport = self.get_config("node", "web.port", None) webport = self.get_config("node", "web.port", None)
if webport: if webport:

View File

@ -926,7 +926,7 @@ class Node(service.MultiService):
NODETYPE = "unknown NODETYPE" NODETYPE = "unknown NODETYPE"
CERTFILE = "node.pem" CERTFILE = "node.pem"
def __init__(self, config, main_tub, control_tub, i2p_provider, tor_provider): def __init__(self, config, main_tub, i2p_provider, tor_provider):
""" """
Initialize the node with the given configuration. Its base directory Initialize the node with the given configuration. Its base directory
is the current directory by default. is the current directory by default.

View File

@ -250,7 +250,6 @@ def create_no_network_client(basedir):
client = _NoNetworkClient( client = _NoNetworkClient(
config, config,
main_tub=None, main_tub=None,
control_tub=None,
i2p_provider=None, i2p_provider=None,
tor_provider=None, tor_provider=None,
introducer_clients=[], introducer_clients=[],

View File

@ -211,7 +211,7 @@ class IntroducerRootTests(SyncTestCase):
main_tub = Tub() main_tub = Tub()
main_tub.listenOn(b"tcp:0") main_tub.listenOn(b"tcp:0")
main_tub.setLocation(b"tcp:127.0.0.1:1") main_tub.setLocation(b"tcp:127.0.0.1:1")
introducer_node = _IntroducerNode(config, main_tub, None, None, None) introducer_node = _IntroducerNode(config, main_tub, None, None)
introducer_service = introducer_node.getServiceNamed("introducer") introducer_service = introducer_node.getServiceNamed("introducer")
for n in range(2): for n in range(2):