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.

Sign up for GitHub

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

Jump to bottom

Allow WebsocketInitFunc to add payload to Ack #4

Merged
merged 3 commits into from
Sep 12, 2023
Merged

Allow WebsocketInitFunc to add payload to Ack #4

merged 3 commits into from
Sep 12, 2023

Commits on Sep 8, 2023

  1. Allow WebsocketInitFunc to add payload to Ack 

    The connection ACK message in the protocol for both
graphql-ws and graphql-transport-ws allows for a payload in the
connection ack message.

We really wanted to use this to establish better telemetry in our use of
websockets in graphql.
    Chris Pride committed Sep 8, 2023
    Configuration menu
    Copy the full SHA
    12afb29 View commit details
    Browse the repository at this point in the history

Commits on Sep 11, 2023

  1. Fix lint error in test

    Chris Pride committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    a1bd2ad View commit details
    Browse the repository at this point in the history

Commits on Sep 12, 2023

  1. Switch argument ordering.

    Chris Pride committed Sep 12, 2023
    Configuration menu
    Copy the full SHA
    a8042fd View commit details
    Browse the repository at this point in the history