Skip to content

Explain that Functions outbound requests leave from shared addresses - #74

Merged
robertbpugh merged 1 commit into
mainfrom
docs/functions-shared-egress
Oct 17, 2026
Merged

robertbpugh merged 1 commit into
mainfrom
docs/functions-shared-egress

Conversation

@robertbpugh

@robertbpugh robertbpugh commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

The Functions page now says that a worker's outbound requests come from shared addresses, so a third-party API can answer 429 on the first call, and lists what to do about it.

Summary

  • Adds a paragraph and four bullets under "Outbound HTTP" on the Functions page.
  • Every space's fetch() leaves through the same Cloudflare host worker. There is no proxy, static IP, or per-space routing in between.
  • An API that rate-limits by IP can count other spaces' traffic against yours, and there is no fixed address to allowlist.
  • The bullets cover options that work today: per-key authentication, retry with backoff, caching in env.DB, and calling CORS-enabled APIs from the browser.
  • Docs only. No product behavior changes.

Testing

  • The checks from AGENTS.md pass locally on Node 24.15.0, Bun 1.3.11, and Vale 3.24.0: verify:generated, check, validate, build, test:docs, audit, verify:public-safety, verify:prose, and verify:routes.

Explain that fetch() from a worker leaves from Cloudflare's shared network
addresses rather than a dedicated or fixed IP per space, so an API that limits
by IP can answer 429 on the first request. List the options that work today:
per-key authentication, retry with backoff, caching in env.DB, and calling
CORS-enabled APIs from the browser.
@robertbpugh robertbpugh self-assigned this Oct 17, 2026
@robertbpugh

Copy link
Copy Markdown
Contributor Author

@indent please review this PR and post any findings as review comments. Please don't push commits to the branch.

@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7f6dd61b-7809-4615-8094-3bb01ae661c6
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@indent

indent Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Issues

No issues found.

CI Checks

All required CI checks passed on 1cb7f46.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No defects: the egress claims match functions-host, and the /crons link and new wording pass.
Full results

Indent Review Agent

  • Summary: No defects: the egress claims match functions-host, and the /crons link and new wording pass.
  • Last ran on commit: 1cb7f461
  • Latest result
    {
      "summary": "No defects: the egress claims match functions-host, and the /crons link and new wording pass.",
      "findings": []
    }

@spacefast
spacefast Bot temporarily deployed to preview/docs/functions-shared-egress October 17, 2026 18:29 Inactive
@indent

indent Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

I've started the review of this PR. I won't push any commits to the branch. Any findings will show up in the review banner.

@indent

indent Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

The review of PR #74 found no defects. The agent checked the shared-egress claims against the Functions host code. It also confirmed the /crons link resolves and that the new wording passes. I have nothing to post as review comments.

@robertbpugh
robertbpugh marked this pull request as ready for review October 17, 2026 19:12
Copilot AI balanced review requested due to automatic review settings October 17, 2026 19:12
@robertbpugh
robertbpugh merged commit 968ada9 into main Oct 17, 2026
6 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch was previously deployed

1 inactive deployment
preview/docs/functions-shared-egress — 1cb7f461 Deployed Oct 17, 2026 by spacefast[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants