[proof of work] feat(keystone): change graphql-playground to GraphiQL #5386
Conversation
|
| new GraphQLApp({ | ||
| apollo: { | ||
| formatError: safeApolloErrorFormatter, | ||
| debug: IS_ENABLE_APOLLO_DEBUG, | ||
| introspection: IS_ENABLE_DANGEROUS_GRAPHQL_PLAYGROUND, | ||
| playground: IS_ENABLE_DANGEROUS_GRAPHQL_PLAYGROUND, | ||
| playground: false, |
There was a problem hiding this comment.
I'm not sure but playground: false might be default
| <head> | ||
| <meta charset="UTF-8"> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
| <title>Playground - __REPLACE_TO_YOUR_SERVER_URL__</title> |
There was a problem hiding this comment.
| <title>Playground - __REPLACE_TO_YOUR_SERVER_URL__</title> | |
| <title>Playground</title> |
There was a problem hiding this comment.
The point here is to be able to differentiate between sandbox tabs of different applications.
This is exactly how it works for us now.
However, we can forget about this moment)
| <script crossorigin src="https://unpkg.com/react-dom/umd/react-dom.production.min.js"></script> | ||
| <script crossorigin src="https://unpkg.com/graphiql/graphiql.min.js"></script> | ||
| <script> | ||
| const apiPath = '__REPLACE_TO_YOUR_API_PATH__' |
There was a problem hiding this comment.
Oh yeah, I found the reason below
There was a problem hiding this comment.
I didn't want to add a library for generating templates, but we already use nunjucks - I'll try it
| headers: { | ||
| 'Content-Type': 'application/json', | ||
| }, | ||
| credentials: 'omit', |
There was a problem hiding this comment.
At the moment we con setup this during front-end. Our partners can use their own tokens
| async prepareMiddleware () { | ||
| // nosemrep: javascript.express.security.audit.express-check-csurf-middleware-usage.express-check-csurf-middleware-usage | ||
| const app = express() | ||
| app.get(this._graphiqlPath, (req, res) => { |
There was a problem hiding this comment.
may be we need to move it to another URL?
No description provided.