search

stratum-mining / stratum

stratum
https://stratumprotocol.org
Other
204 stars 121 forks source link

TProxy Config for Client Timeout #413

Open coleFD opened 1 year ago

coleFD commented 1 year ago

Related to SHUTDOWN_TIMEOUT_SECS in #405. If #405 gets merged before integrating the config value, make sure remove SHUTDOWN_TIMEOUT_SECS and integrate the timeout logic used in the downstream.

For certain scenarios when a downstream client connects to the TProxy and they are not initialize to receive jobs, no other messages are sent on the connection, and therefore, the TProxy will never know when to terminate the connection. For this issue, we need to add a downstream_timeout config to the proxy-config.toml

pavlenex commented 1 year ago

@coleFD Since #405 got merged, how do we proceed with this one?

coleFD commented 1 year ago

@coleFD Since #405 got merged, how do we proceed with this one?

We need to move the constant defined in the downstream/mod.rs file to the proxy-config.toml. It is low priority since it's just meant to allow the value to be defined by the dev