jsonrpc_err_in_rent #41

goodboy · 2025-02-21T21:17:05Z

goodboy commented

2025-02-21 21:17:05 +00:00

Lel, redo of #36 after I forgot to change the dst branch prior to merge..

Orig content from #36

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.

Lel, redo of #36 after I forgot to change the dst branch prior to merge.. #### Orig content from #36 --- 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-21 21:17:06 +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-21 21:20:17 +00:00

subsys_refinery #29

goodboy merged commit 5e371f1d73 into gitea_feats

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

goodboy referenced this issue from a commit

2025-02-21 21:21:03 +00:00

Merge pull request 'jsonrpc_err_in_rent' (#41) from jsonrpc_err_in_rent into gitea_feats

Sign in to join this conversation.

No reviewers

No Label

No Milestone