Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for SendLobbyPacket and non-IPC segment types #15

Merged
merged 3 commits into from
Mar 13, 2023
Merged

Conversation

ff14wed
Copy link
Owner

@ff14wed ff14wed commented Mar 13, 2023

This change adds the ability to receive the Lobby channel for Send and segments that are not of type SEGMENTTYPE_IPC.

There are a few behavioral changes as a result of this change, though
they should not be breaking for standard use cases.

  1. Deucalion now accepts a sig for SendLobbyPacket via sending a Payload
    with OP Send and CHANNEL 0 (Lobby). Deucalion accepts a sig for
    SendPacket via a Payload with OP Send and any CHANNEL != 0.
  2. The SERVER HELLO message has been shortened like so:
    SERVER HELLO. HOOK STATUS: RECV {}. SEND {}. SEND_LOBBY {}.
  3. Deucalion now emits two new OP types: RecvOther and SendOther. Subscribers can opt-into receiving them with the Option flag for allowing other packets (1 << 6).

ff14wed added 3 commits March 12, 2023 18:57
Previously this was not implemented.
There are a few behavioral changes as a result of this change, though
they should not affect a normal use case.

1. Deucalion now accepts a sig for SendLobbyPacket via sending a Payload
   with OP `Send` and CHANNEL `0` (Lobby). Deucalion accepts a sig for
   SendPacket via a Payload with OP `Send` and any CHANNEL != 0.
2. The SERVER HELLO message has been shortened like so:
   `SERVER HELLO. HOOK STATUS: RECV {}. SEND {}. SEND_LOBBY {}.`
This change adds two new OPs for the protocol, one for received non-IPC
segments, and one for sent non-IPC segments. The same AllowOther Option
bitflag will allow for receiving this data on a subscriber.
@ff14wed ff14wed merged commit b13baa7 into main Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant