Skip to content

Conversation

@patriciasantaana
Copy link
Contributor

Summary

Updates description/headers for rate limiting best practices on graphql

Documentation checklist

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Jul 17, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Updated GraphQL rate limiting documentation to enhance clarity and best practices for preventing server overload.

  • Revised the introduction to emphasize the challenges of rate limiting in GraphQL compared to RESTful APIs.
  • Added recommendations for limiting user requests based on operation name and query complexity.
  • Renamed sections to better reflect their content regarding query complexity limits.

Modified Files

  • src/content/docs/waf/rate-limiting-rules/best-practices.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/waf/ @pedrosousa, @cloudflare/firewall, @cloudflare/pcx-technical-writing

@github-actions
Copy link
Contributor

github-actions bot commented Jul 17, 2025

@patriciasantaana patriciasantaana marked this pull request as ready for review July 17, 2025 22:09
@patriciasantaana patriciasantaana requested review from a team and pedrosousa as code owners July 17, 2025 22:09
@patriciasantaana patriciasantaana enabled auto-merge (squash) July 18, 2025 15:51
@patriciasantaana patriciasantaana merged commit b0554c0 into production Jul 18, 2025
8 checks passed
@patriciasantaana patriciasantaana deleted the patricia/pcx18400-graphql-mod branch July 18, 2025 16:53
sdnts pushed a commit to sdnts/cloudflare-docs that referenced this pull request Jul 24, 2025
* graphql rate limiting updates

* Apply suggestion from @pedrosousa

Co-authored-by: Pedro Sousa <[email protected]>

---------

Co-authored-by: Pedro Sousa <[email protected]>
thomasgauvin pushed a commit that referenced this pull request Aug 15, 2025
* graphql rate limiting updates

* Apply suggestion from @pedrosousa

Co-authored-by: Pedro Sousa <[email protected]>

---------

Co-authored-by: Pedro Sousa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants