-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
IntrospectionQuery fails, unable to run graphiql #2085
Comments
What version of graphql-js are you using on the server? My bad, the useDeprecated option might not be compatible with lower than graphql@16. This is something I just added. your server does not recognize this new introspection option because it isn’t running a version that supports it. I will need to do some kind of feature detection. |
@saihaj or @IvanGoncharov should we make this opt-in for the user, or is there some way we are able to detect if it’s safe to add this argument to introspection queries? |
Thanks for looking into the issue. Unfortunately, I am unable to find the graphql-js version that the project is currently using |
I reverted to https://unpkg.com/[email protected]/* as a work-around until this is fixed. |
Oh thank god I'm not the only one. I thought I was going crazy. This is really inconvenient too; I have a demo of our new Netflix DGS service scheduled for tomorrow and just found out GraphiQL broke. Is there a workaround for this? If I can't get this to work tomorrow morning it's going to be quite embarrassing. |
@nielsutrecht See my reply. #2085 (comment) Revert to the 1.5.8 unpkg.com distribution. |
@blevine thanks but unfortunately it's a transient dependency. Netflix DGS links to the latest production release. I'll go see if I can figure something out on my end. I'll probably just use a stand-alone tool for the demo :) |
@acao It would be great of you could revert this change and publish to unpkg.com until you have a fix. |
fixed! |
@acao How long before the fix gets to unpkg.com? |
I see 1.5.9 is on npm, so that's a question for unpkg! |
I will check into this after work in a few hours, today is a busy day. We're hoping to add a test matrix to test against older versions of |
@acao 1.5.9 has the breaking change. |
For the record, the fix appears to be in 1.5.11 which was published to npm and is also published to unpkg.com |
lol yes sorry. npmjs.com was so cached in my browser, it didn't work until i pointed to 1.5.11 specifically 🤣 managing 2 dozen graphql packages, can't always keep track of versions honestly. need an atlassian changesets automation or custom script to post comments for fix releases. or heck we could just use semantic release. I couldn't get changesets to support pre-releases today without putting every single package in pre-release, which wasn't necessary at all :/ |
We are getting a slightly different
I tried pinning to 1.5.11 but it's still broken. I'm going to try downgrading to 1.5.8 and see if that fixes it. |
@myronmarston your graphql schema needs to support I'm preparing a fix that makes this an opt-in feature, as I wasn't aware that so many graphql schemas do not support this feature on introspection yet. |
Hi everyone,
as of this morning (but possibly even earlier) I haven't been able to run graphiql in my project anymore. After extensive investigation, I've come to a conclusion that none of my changes have broken the app.
It seems to me that the IntrospectionQuery has changed and is not compatible anymore.
Here is the IntrospectionQuery that it's trying to run:
and here are the errors that I'm getting:
I am using:
And I was able to get the following link from the console:
https://unpkg.com/[email protected]/graphiql.min.js
Let me know if I can provide any more information regarding this issue.
Thank you!
The text was updated successfully, but these errors were encountered: