Skip to content

Move remote request handling into core behind the remote-adapter feature flag #774

Description

@LiamCarPer

Part of #727.

Move the duplicated remote-request handling from the redis, postgres and mongodb
adapters into socketioxide_core behind the existing remote-adapter feature flag.

Current state: each adapter implements a near-identical recv_req dispatch plus the
same recv_* handlers (~110 LOC per adapter): broadcast, broadcast_with_ack,
disconnect_sockets, all_rooms, add/del sockets, fetch sockets. Each also repeats the
same get_res / broadcast_with_ack skeleton (response channel setup,
ResponseHandlers insert, ResponseTypeId filter, take(remote_serv_cnt), request
timeout).

Proposed shared API (sketch, in core):

Each adapter then keeps only its wire-specific parts (codec, driver calls,
channel/item routing). No behavior or wire-format change.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions