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

Fix SIGSEGV when client subs to multiple fields #398

Merged

pavelnikolov merged 1 commit into graph-gophers:master from nmaquet:master

Jun 17, 2020

Contributor

nmaquet commented Jun 13, 2020

This fixes a SIGSEGV panic which can happen when a client attempts to subscribe to more than one field.


          Fix SIGSEGV when client subs to multiple fields

c80e625

Contributor Author

nmaquet commented Jun 17, 2020

@pavelnikolov Would you consider merging this? It's a simple fix and it's all tested and ready to go :)

pavelnikolov approved these changes

View reviewed changes

pavelnikolov merged commit 83ba819 into graph-gophers:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet