juno icon indicating copy to clipboard operation
juno copied to clipboard

Move transport-specific handlers to sub-packages in `jsonrpc`

Open joshklop opened this issue 1 year ago • 3 comments

jsonrpc implements the (transport-agnostic) JSON-RPC v2 spec. Dependents should not need to rely on transport code in net/http and nhooyr.io/websocket without explicit requesting it.

joshklop avatar Aug 03 '23 03:08 joshklop

Is this still relevant?

omerfirmak avatar Dec 01 '23 14:12 omerfirmak

Yes. I was planning on doing this after #1389, but ended up focusing on other things in the meantime.

joshklop avatar Dec 02 '23 00:12 joshklop

Noticed while working on this: we are using the same namespace, subsystem, and name for all RPC method counters, across all servers. Do we need to aggregate them somehow?

joshklop avatar Dec 02 '23 23:12 joshklop