KEMBAR78
Improve potential errors thrown by `startRemoteProxySession` by including more information by dario-piotrowicz · Pull Request #11018 · cloudflare/workers-sdk · GitHub
Skip to content

Conversation

@dario-piotrowicz
Copy link
Member

I noticed that when I try to run wrangler/vite dev in non-interactive mode, since I have more than one account associated to my user, starting a remote proxy session fails since wrangler doesn't know what account to use. I think that this is acceptable, that's problematic is however that this information is completely missing in the terminal:
Screenshot 2025-10-19 at 17 19 27

So here I am improving the error thrown by startRemoteProxySession to also include the missing information:
Screenshot 2025-10-19 at 17 17 58


  • Tests
    • Tests included
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: DX improvement
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: not a v3 feature

@dario-piotrowicz dario-piotrowicz requested a review from a team as a code owner October 19, 2025 16:23
@changeset-bot
Copy link

changeset-bot bot commented Oct 19, 2025

🦋 Changeset detected

Latest commit: 6623fb6

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

Failed to automatically backport this PR's changes to Wrangler v3. Please manually create a PR targeting the v3-maintenance branch with your changes. Thank you for helping us keep Wrangler v3 supported!

Depending on your changes, running git rebase --onto v3-maintenance main dario/start-remote-proxy-surface-error might be a good starting point.

Notes:

  • your PR branch should be named v3-backport-11018
  • add the skip-v3-pr label to the current PR to stop this workflow from failing

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 19, 2025

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@11018

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@11018

miniflare

npm i https://pkg.pr.new/miniflare@11018

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@11018

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@11018

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@11018

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@11018

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@11018

wrangler

npm i https://pkg.pr.new/wrangler@11018

commit: 6623fb6

@dario-piotrowicz dario-piotrowicz added the skip-v3-pr Skip validation of presence of a v3 backport PR label Oct 19, 2025
@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Oct 20, 2025
@petebacondarwin petebacondarwin merged commit 5124818 into main Oct 20, 2025
45 of 49 checks passed
@petebacondarwin petebacondarwin deleted the dario/start-remote-proxy-surface-error branch October 20, 2025 14:25
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-v3-pr Skip validation of presence of a v3 backport PR

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants