-
Notifications
You must be signed in to change notification settings - Fork 327
Upgrade Graphiql to enable headers #441
Comments
It's finally resoved by graphql/graphiql#1543, this is good. |
We welcome PRs if anybody has the time to implement this. |
I've upgraded GraphiQL to 1.0.6. Will be available in release 8.1.0. To enable the headers you can add the following application property: graphiql:
props:
variables:
headerEditorEnabled: true Just like you can configure any of the GraphiQL Props under the |
Although the headers editor was present (graphql-java-kickstart#441), it was non-functional because only the preconfigured headers were submitted, regardless of the content of the request headers tab.
Although the headers editor was present (graphql-java-kickstart#441), it was non-functional because only the preconfigured headers were submitted, regardless of the content of the request headers tab.
Apart of the issue addressed in the PR above, I noticed several issues with the header configuration:
@oliemansm Since this is an unreleased feature, I am not sure if a new issue should be raised or this one should be reopened. |
Note: You'll get an error about trying to convert a boolean to a string if you're running an old version of |
Is your feature request related to a problem? Please describe.
Currently, the GraphiQL SpringBoot starter does not support adding dynamic headers.
Describe the solution you'd like
With the newest version of GraphiQL, a new tab was added which allows for dynamic headers.
graphql-spring-boot
should use that latest version. See the GraphiQL issue provided under additional context.Describe alternatives you've considered
The alternative is to set the header in the application config, but this is a static header. Changing headers requires a restart.
Additional context
graphql/graphiql#59
The text was updated successfully, but these errors were encountered: