jsonrpc_err_in_rent: much needed refinements to .data._web_bs.open_jsonrpc_session() #36

goodboy · 2025-02-20T16:25:20Z

goodboy commented

2025-02-20 16:25:20 +00:00

Namely ensuring error propagation to the parent trio.Task as well as cleaning out old unused parameters and exposing the underlying wy timeout to avoid breaking non-chatty sessions (like for options mkts)!

Deats,

a slight rework to the .data._web_bs.open_jsonrpc_session() to ensure any recved IPC msg errors are relayed to and raised in the parent task.
doc-string and cleanup of old params.
making the underlying websocket timeout a float with default float('inf') to avoid timeout-caused connection resets which break rpc sesh.

Namely ensuring error propagation to the parent `trio.Task` as well as cleaning out old unused parameters and exposing the underlying wy timeout to avoid breaking non-chatty sessions (like for options mkts)! #### Deats, - a slight rework to the `.data._web_bs.open_jsonrpc_session()` to ensure any recved IPC msg errors are relayed to and raised in the parent task. - doc-string and cleanup of old params. - making the underlying websocket timeout a `float` with default `float('inf')` to avoid timeout-caused connection resets which break rpc sesh.

goodboy added 3 commits 2025-02-20 16:25:20 +00:00

76735189de data._web_bs: try to raise jsonrpc errors in parent task

5633f5614d Doc-n-clean `.data._web_bs.open_jsonrpc_session()`

Add a doc-string reflecting recent refinements, drop all the old hook
params, rename `n: trio.Nursery` -> `tn` for "task nursery" fitting with
code base's naming style.

e391c896f8 Mk jsronrpc's underlying ws timeout `float('inf')`

Since currently we're only using this IPC subsys for `deribit`, and
generally speaking we're primarly supporting options markets (which are
fairly "slow moving"), flip to a default of NOT resetting the `NoBsWs`
on timeout since doing so normally breaks the jsron-rpc IPC session.
Without a proper `fixture` passed to `open_autorecon_ws()` (which we
should eventually implement!!) relying on a timeout-to-reset more or
less will just cause breakage issues - a proper reconnect sequence must
be implemented before using that feature.

Deats,
- expose and proxy through the `msg_recv_timeout` from
  `open_jsonrpc_session()` into the underlying `open_autorecon_ws()`
  call.

goodboy referenced this pull request

2025-02-20 16:25:44 +00:00

subsys_refinery #29

goodboy merged commit f3a14599fc into tsp_gaps

2025-02-21 21:11:09 +00:00

goodboy referenced this issue from a commit

2025-02-21 21:11:11 +00:00

Merge pull request 'jsonrpc_err_in_rent: much needed refinements to `.data._web_bs.open_jsonrpc_session()`' (#36) from jsonrpc_err_in_rent into tsp_gaps

goodboy commented

2025-02-21 21:12:23 +00:00

Poster

Lol well then, forgot to change the dst branch before doing that smh..