add support for graphql-core 3 to graphql_ws.aiohttp

[hoefling]

Signed-off-by: oleg.hoefling oleg.hoefling@gmail.com

This PR adds support for graphql-core>=3 to graphql.aiohttp.AiohttpSubscriptionServer. Note that this will break the rest of the BaseSubscriptionServer implementations! Unfortunately, I am not familiar with Sanic or gevent to adapt those too.

This PR violates against the PR submission rule

3 The pull request should work for Python 2.6, 2.7, 3.3, 3.4 and 3.5, and for PyPy. Check https://travis-ci.org/graphql-python/graphql_ws/pull_requests and make sure that the tests pass for all supported Python versions.

as graphql-core>=3 is compatible with Python 3.6 onwards. Also, all of the Python versions listed have reached EOL anyway (except Python 3.5 which will reach EOL in September).

[codecov-io]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[bueltan]

Thank for sharing , it's really appreciated.

[betaboon]

the connection_context.get_operation(op_id).dispose() seems to depend on the old rxpy-dependency that has been removed from graphql-core and is effectively removed in this branch by removing setup_observable_extension.

this causes an unhandled exception on unsubscribe

[dopry]

@hoefling do you have any plans to update this PR?

The wider ecosystem seems to be adopting graphql-core 3 and the version constraint here is bound to start generating issues. I'm experiencing issues in the examples for https://github.com/torchbox/wagtail-grapple since we've updated to django-graphene >= 3.

[hoefling]

@dopry let me take a look at it, it's been a while now :-)

[erikwrede]

Any objections against just integrating this into GQL-Server? Websocket-Based Subscriptions have become a common pattern in GQL and an integration would reduce the amount of duplicate code and management effort.