Skip to content

[Fundamentals] Clarify connection terminology in connection-limits reference - #34522

Merged
irvinebroque merged 1 commit into
productionfrom
support/connection-limits-tcp-disambiguation
Oct 22, 2026
Merged

irvinebroque merged 1 commit into
productionfrom
support/connection-limits-tcp-disambiguation

Conversation

@ngayerie

Copy link
Copy Markdown
Collaborator

Clarify what "connection" means in the context of the origin-server timeout rows.

  • Add "What connection means here" section explaining TCP vs HTTPS setup (TCP handshake complete ≠ HTTPS ready)
  • Rename "Complete TCP Connection" → "TCP connection establishment"
  • Rename "TCP ACK Timeout" → "TCP request acknowledgment timeout"
  • Note that neither timeout row is an exhaustive list of operations that can produce a 522, and neither value is a universal TCP+TLS setup budget

Split from #33662 as this file has different reviewers.

…ference

Clarify what "connection" means in the context of the origin-server
timeout rows, following review from dpolaske-cloudflare.

- Add "What connection means here" section explaining TCP vs HTTPS setup
- Rename "Complete TCP Connection" to "TCP connection establishment"
- Rename "TCP ACK Timeout" to "TCP request acknowledgment timeout"
- Note that neither row is an exhaustive list of operations that can
  produce 522, and neither value is a universal TCP+TLS setup budget
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

AI Review

✅ Reviewed 2a4200d · 0 findings · incremental

Commands
  • /review Run a review now.
  • /full-review Review the entire PR diff.
  • /disable-auto-review Stop automatic reviews.
  • /rebase Rebase against production.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://support-connection-limits-tcp-disambiguation.previews.developers.cloudflare.com (commit 2a4200d)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://46d3c43e.previews.developers.cloudflare.com 2a4200d 2026-10-22T08:17:37.125Z Visit the dashboard ↗

@irvinebroque
irvinebroque added this pull request to the merge queue Oct 22, 2026
Merged via the queue into production with commit 9696646 Oct 22, 2026
15 checks passed
@irvinebroque
irvinebroque deleted the support/connection-limits-tcp-disambiguation branch October 22, 2026 23:42
ngayerie added a commit that referenced this pull request Oct 23, 2026
Expand the 499 troubleshooting page based on Q3 2026 support case analysis.
Addresses all review comments from dpolaske-cloudflare.
connection-limits.mdx changes are split into PR #34522.
lambrospetrou pushed a commit to lambrospetrou/cloudflare-docs that referenced this pull request Oct 23, 2026
Expand the 499 troubleshooting page based on Q3 2026 support case analysis.
Addresses all review comments from dpolaske-cloudflare.
connection-limits.mdx changes are split into PR cloudflare#34522.
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