Skip to content

fix(cloudflare): fetch tunnel connections from dedicated endpoint after API removal - #8624

Open
The-AarushiSingh wants to merge 5 commits into
simstudioai:stagingfrom
The-AarushiSingh:fix/cloudflare-tunnel-connections-v2
Open

The-AarushiSingh wants to merge 5 commits into
simstudioai:stagingfrom
The-AarushiSingh:fix/cloudflare-tunnel-connections-v2

Conversation

@The-AarushiSingh

Copy link
Copy Markdown

fixes #8616

Supersedes #8623 (that PR targeted main and included unrelated commits).

Cloudflare removed the connections array from cfd_tunnel get/list responses (EOL 2026-10-05).

  • get_tunnel: secondary GET .../cfd_tunnel/{id}/connections and flatMap connector.conns
  • list_tunnels: connections always null (avoids N+1)
  • secondary fetch respects abort signal when provided

@vercel

vercel Bot commented Oct 5, 2026

Copy link
Copy Markdown

@The-AarushiSingh is attempting to deploy a commit to the Sim Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps

greptile-apps Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Updates Cloudflare tunnel API integration to match endpoint changes.

The PR appears safe to merge; no new actionable issue was identified.

Summary

The PR fetches tunnel connections from Cloudflare’s dedicated endpoint and leaves connections null in list results to avoid per-tunnel requests.

  • The change since the previous review adds the required annotation to the external fetch.

Reviews (3) · Last reviewed commit: "chore(cloudflare): annotate raw fetch to..."

Comment thread apps/sim/tools/cloudflare/get_tunnel.ts Outdated
Comment thread apps/sim/tools/cloudflare/get_tunnel.ts
@The-AarushiSingh
The-AarushiSingh changed the base branch from main to staging October 5, 2026 13:18
@The-AarushiSingh

Copy link
Copy Markdown
Author

@greptileai review

Comment thread apps/sim/tools/cloudflare/get_tunnel.ts
@The-AarushiSingh

Copy link
Copy Markdown
Author

@greptileai review

@The-AarushiSingh

Copy link
Copy Markdown
Author

@cubic review

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

@cubic review

@The-AarushiSingh I can’t start this review because this PR was opened by an external contributor in a public repository. Ask an installation member or outside collaborator to comment @cubic review this.

This branch has not been deployed

No deployments
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.

[BUG] Cloudflare get_tunnel / list_tunnels always return connections: null since 2026-10-05

1 participant