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

Subscription broker #274

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Subscription broker #274

wants to merge 3 commits into from

Commits on Sep 21, 2022

  1. Add subscription broker callback 

    Previously subscriptions over HTTP were handled through SSE.
This commits adds a callback so the user can choose to handle it
differently. It's modelled after how [graphl-ruby](https://graphql-ruby.org/subscriptions/pusher_implementation.html)
does it.

I think this gives the flexibility to the user to implement
their own subscription broker implementation. It gives access to the
http request to be changed in a way the user sees fit.
There's still some work to be done on the Absinthe side for a user,
though this can be done in userland by replacing some phases.

`absinthe_phoenix` could use a similar callback.

See absinthe-graphql#217 for
previous work.
    @maartenvanvliet
    maartenvanvliet committed Sep 21, 2022
    Configuration menu
    Copy the full SHA
    a1e6c48 View commit details
    Browse the repository at this point in the history

  2. Add some info on pipeline config

    @maartenvanvliet
    maartenvanvliet committed Sep 21, 2022
    Configuration menu
    Copy the full SHA
    9b39bf0 View commit details
    Browse the repository at this point in the history

  3. Use ExUnit start_supervised! processes

    @maartenvanvliet
    maartenvanvliet committed Sep 21, 2022
    Configuration menu
    Copy the full SHA
    8b4aa07 View commit details
    Browse the repository at this point in the history