Skip to content

Conversation

@dario-piotrowicz
Copy link
Member

@dario-piotrowicz dario-piotrowicz commented Jul 30, 2025

Fixes #9916
Fixes https://jira.cfdata.org/browse/DEVX-2141

There are three related changes/fixes in this PR:

  • fixing the fact that getPlatformProxy is currently not taking into account specified environments for remote bindings
  • fixing the same issue in vitest-pool-workers
  • updating the maybeStartOrUpdateRemoteProxySession function so that callers can specify environments (this enables the abovementioned two fixes, since both getPlatformProxy and vitest-pool-workers use this function)

For more details check out the changesets 🙂


@dario-piotrowicz dario-piotrowicz requested a review from a team as a code owner July 30, 2025 11:46
@dario-piotrowicz dario-piotrowicz added the skip-v3-pr Skip validation of presence of a v3 backport PR label Jul 30, 2025
@changeset-bot
Copy link

changeset-bot bot commented Jul 30, 2025

🦋 Changeset detected

Latest commit: 1d7c840

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

This PR includes changesets to release 3 packages
Name Type
wrangler Patch
@cloudflare/vitest-pool-workers Patch
@cloudflare/vite-plugin Patch

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jul 30, 2025

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: 1d7c840

@dario-piotrowicz dario-piotrowicz force-pushed the dario/DEVX-2141/maybeStartRemoteSession-ignoring-env-fix branch from cb072d0 to 1d7c840 Compare August 5, 2025 10:34
@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Aug 5, 2025
@dario-piotrowicz dario-piotrowicz merged commit 773cca3 into main Aug 5, 2025
43 of 50 checks passed
@dario-piotrowicz dario-piotrowicz deleted the dario/DEVX-2141/maybeStartRemoteSession-ignoring-env-fix branch August 5, 2025 16:15
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Aug 5, 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

Archived in project

Development

Successfully merging this pull request may close these issues.

Worker run fails to respect environment when using with platform proxy

2 participants