GraphQLBundle icon indicating copy to clipboard operation
GraphQLBundle copied to clipboard
verified publisher icon youshido-php

GraphiQL tweaks

Open bjunc opened this issue 8 years ago • 2 comments

Few suggestions I'd like to make:

  1. Change the default tokenHeader from access-token to Authorization. I believe the majority of use-cases will be using Authorization: Bearer {JWT}.

  2. Remove the @Cache from the explorer controller. Is this really needed for dev?

  3. Change the route path to something that doesn't start with /graphql. I have /graphql behind a firewall pattern, which then blocks the explorer too. I've since changed the path to /_graphiql to match other dev related tools (eg. _profiler)

  4. Make the explorer a dedicated bundle that can added to the AppKernel based on environment (eg. DebugBundle and WebProfilerBundle).

bjunc avatar Jun 19 '17 13:06 bjunc

Hi @bjunc,

I would like to add some notes:

  1. Let's make it as config parameter (of new budle).
  2. Yes, you can easily remove it, it's not necessary any more.
  3. Yes too.
  4. Absolutely yes. It would be great to add here else one extension for graphql Altair.

portey avatar Jun 19 '17 14:06 portey

Sounds great! @portey what are the next steps?

bjunc avatar Jun 19 '17 18:06 bjunc