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.
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
Fixes graphiql so it works with koa #370
Conversation
When run with koa, static files for graphiql would fail with error message `Error: Can't set headers after they are sent.` This is caused by pillarjs/send being incompatible with koa, at least in the manner it was used here. See pillarjs/send#118 and pillarjs/send#119 for more discussion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me; thanks! 👍
For everybody who is waiting for the pull request to land you can work around this issue with koa-connect. Simply wrap the |
I've rebuilt a few times and cleared the relevant cache but it still fails on an npm issue. |
Seems to be unrelated to the changes here, built locally and got the same error both for master branch and this branch (node v4.3.2 and npm v2.14.12). Looks like it is caused by graphiql@0.8.0 not happy with graphql versions >= 0.9. Any reason we can't update the graphiql dependency to follow graphql? |
I defer to @calebmer on that; he's got a better grasp of the various versions and why they were chosen than I do. |
CI was passing when I just looked. Updated the branch, and if CI still passes we can merge 😊 |
Released in 3.1.0 🎉 |
* Fixes graphiql so it works with koa When run with koa, static files for graphiql would fail with error message `Error: Can't set headers after they are sent.` This is caused by pillarjs/send being incompatible with koa, at least in the manner it was used here. See pillarjs/send#118 and pillarjs/send#119 for more discussion * Removed unnecessary semicolon to make linter happy
When run with koa, static files for graphiql would fail with error
message
Error: Can't set headers after they are sent.
This is caused by pillarjs/send being incompatible with koa, at least in
the manner it was used here. See pillarjs/send#118 and pillarjs/send#119
for more discussion