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

Ariadne graphql and graphql_sync will attempt to execute subscription operations #1164

Closed
rafalp opened this issue Feb 21, 2024 · 0 comments · Fixed by #1165
Closed

Ariadne graphql and graphql_sync will attempt to execute subscription operations #1164

rafalp opened this issue Feb 21, 2024 · 0 comments · Fixed by #1165
Assignees
@rafalp
Labels
bug Something isn't working
Milestone
0.23

Comments

@rafalp
Copy link
Contributor

rafalp commented Feb 21, 2024

graphql and graphql_sync should check if operation to execute is subscription, and prevent it if so. We can build on logic from #1160 in this
@rafalp rafalp added the bug Something isn't working label Feb 21, 2024
@rafalp rafalp added this to the Next release milestone Feb 21, 2024
@rafalp rafalp self-assigned this Feb 21, 2024
@rafalp rafalp mentioned this issue Feb 21, 2024
Merged
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rafalp rafalp

Labels
bug Something isn't working
Projects
None yet
Milestone
0.23
Development

Successfully merging a pull request may close this issue.

Prevent subscription execution without websocket mirumee/ariadne
1 participant
@rafalp