Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(backend): helmet + graphiql #6303

Merged
merged 4 commits into from
May 16, 2023
Merged

fix(backend): helmet + graphiql #6303

merged 4 commits into from
May 16, 2023

Conversation

ulfgebhardt
Copy link
Member

@ulfgebhardt ulfgebhardt commented May 15, 2023

🍰 Pullrequest

Since the update to helmet 7.x (former 3.x) helmet checks cross origin. Apparently the graphQL Playground loads its complete script from a CDN. Hence this rule must be disabled in DEBUG mode(where the Playground is available)

Issues

Todo

@ulfgebhardt ulfgebhardt merged commit fef12f5 into master May 16, 2023
@ulfgebhardt ulfgebhardt deleted the fix-graphiql-helmet branch May 16, 2023 10:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants