Skip to content

containers: Remove experimental flag from enabling egress interception for containers#12826

Merged
edmundhung merged 2 commits intomainfrom
gv/experimental-out
Mar 10, 2026
Merged

containers: Remove experimental flag from enabling egress interception for containers#12826
edmundhung merged 2 commits intomainfrom
gv/experimental-out

Conversation

@gabivlj
Copy link
Contributor

@gabivlj gabivlj commented Mar 10, 2026

We were backing this feature from local dev via a experimental flag. This enables the usage of container.interceptOutboundHttp for everybody!

(Wednesday 2026-03-11 is the release date in production).


A picture of a cute animal (not mandatory, but encouraged)
image


Open with Devin

@gabivlj gabivlj requested review from a team as code owners March 10, 2026 02:31
@changeset-bot
Copy link

changeset-bot bot commented Mar 10, 2026

🦋 Changeset detected

Latest commit: 86bb456

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

@workers-devprod
Copy link
Contributor

workers-devprod commented Mar 10, 2026

Codeowners approval required for this PR:

  • ✅ @cloudflare/cloudchamber
  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@github-actions
Copy link
Contributor

github-actions bot commented Mar 10, 2026

✅ All changesets look good

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 10, 2026

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: e8d2bdf

@gabivlj gabivlj force-pushed the gv/experimental-out branch from 4ae073c to 621f95d Compare March 10, 2026 02:48
@emily-shen
Copy link
Contributor

emily-shen commented Mar 10, 2026

FYI CI was failing and I pushed a fix:

  • remove experimental flag from the e2e dev test. that makes the new test in containers-shared redundant, so i just removed that since it was failing CI on windows anyway

and then since i was in the area, also now export the DEFAULT_CONTAINER_EGRESS_INTERCEPTOR_IMAGE from containers-shared instead of including it in two places.

and verified it works with the vite plugin :)

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Mar 10, 2026
gabivlj and others added 2 commits March 10, 2026 12:54
…n for containers

We were backing this feature from local dev via a experimental flag.
This enables the usage of `container.interceptOutboundHttp` for
everybody!

(Wednesday 2026-03-11 is the release date in production).
@edmundhung edmundhung force-pushed the gv/experimental-out branch from 01308a8 to 86bb456 Compare March 10, 2026 12:54
@edmundhung edmundhung merged commit de65c58 into main Mar 10, 2026
43 of 47 checks passed
@edmundhung edmundhung deleted the gv/experimental-out branch March 10, 2026 14:20
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Mar 10, 2026
@gabivlj
Copy link
Contributor Author

gabivlj commented Mar 10, 2026

Thanks so much @emily-shen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants