Skip to content

fix specified environments being ignored for remote bindings in some cases#10130

Merged
dario-piotrowicz merged 1 commit intomainfrom
dario/DEVX-2141/maybeStartRemoteSession-ignoring-env-fix
Aug 5, 2025
Merged

fix specified environments being ignored for remote bindings in some cases#10130
dario-piotrowicz merged 1 commit intomainfrom
dario/DEVX-2141/maybeStartRemoteSession-ignoring-env-fix

Conversation

@dario-piotrowicz
Copy link
Copy Markdown
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
@changeset-bot
Copy link
Copy Markdown

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
Copy Markdown

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

None yet

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

3 participants