Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(graphql): enable configurable max_query_depth and max_query_complexity #2128

Merged
merged 2 commits into from
Jan 15, 2025
Merged

feat(graphql): enable configurable max_query_depth and max_query_complexity #2128

merged 2 commits into from
Jan 15, 2025

Conversation

vinceAmstoutz
Copy link
Contributor

@vinceAmstoutz vinceAmstoutz commented Jan 15, 2025 •
edited
Loading

Replaces #2107 to document api-platform/core#6880 with Laravel support and some improvements.

PR started by @mauriau

@vinceAmstoutz vinceAmstoutz mentioned this pull request Jan 15, 2025
Closed
@vinceAmstoutz vinceAmstoutz changed the base branch from 4.0 to main January 15, 2025 12:31
@vinceAmstoutz
Copy link
Contributor Author

@soyuka Could you, please, take a look at this PR which is related to the code in PR #6880 that you merged earlier?

@vinceAmstoutz vinceAmstoutz self-assigned this Jan 15, 2025
@vinceAmstoutz vinceAmstoutz merged commit f13161a into api-platform:main Jan 15, 2025
1 check passed
@vinceAmstoutz vinceAmstoutz deleted the feat/graphql-security branch January 15, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@soyuka soyuka soyuka approved these changes

@alanpoulain alanpoulain Awaiting requested review from alanpoulain

Assignees

@vinceAmstoutz vinceAmstoutz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vinceAmstoutz @soyuka @mauriau