Skip to content

[vite-plugin] Close disabled HMR WebSocket upgrades - #15290

Open
edmundhung wants to merge 1 commit into
mainfrom
edmundhung/fix-vite-disabled-websocket
Open

[vite-plugin] Close disabled HMR WebSocket upgrades#15290
edmundhung wants to merge 1 commit into
mainfrom
edmundhung/fix-vite-disabled-websocket

Conversation

@edmundhung

Copy link
Copy Markdown
Member

Fixes #15285.

Close Vite HMR upgrade sockets when server.ws is disabled and no Vite WebSocket handler can claim them. Continue delegating requests to Vite when enabled, while preserving Worker and Sandbox WebSocket forwarding.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: This restores documented Vite server.ws: false behavior and adds no new user-facing API.

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

@changeset-bot

changeset-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b526abf

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

This PR includes changesets to release 1 package
Name Type
@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

@ask-bonk

ask-bonk Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

github run

@ask-bonk

ask-bonk Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@edmundhung Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

@github-actions

Copy link
Copy Markdown
Contributor

✅ All changesets look good

@pkg-pr-new

pkg-pr-new Bot commented Aug 20, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@15290

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@15290

@cloudflare/codemods

npm i https://pkg.pr.new/@cloudflare/codemods@15290

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@15290

create-cloudflare

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

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@15290

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@15290

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-plugin

npm i https://pkg.pr.new/@cloudflare/vitest-plugin@15290

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@15290

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@15290

wrangler

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

commit: b526abf

@edmundhung
edmundhung force-pushed the edmundhung/fix-vite-disabled-websocket branch 2 times, most recently from 3472681 to dcd5dd7 Compare August 20, 2026 18:45
@edmundhung
edmundhung force-pushed the edmundhung/fix-vite-disabled-websocket branch from dcd5dd7 to b526abf Compare August 21, 2026 16:59
@edmundhung
edmundhung marked this pull request as ready for review August 21, 2026 17:20
@workers-devprod
workers-devprod requested review from a team and penalosa and removed request for a team August 21, 2026 17:20
@workers-devprod

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • .changeset/calm-websockets-close.md: [@cloudflare/wrangler]
  • packages/vite-plugin-cloudflare/src/tests/websockets.spec.ts: [@cloudflare/wrangler]
  • packages/vite-plugin-cloudflare/src/plugins/dev.ts: [@cloudflare/wrangler]
  • packages/vite-plugin-cloudflare/src/websockets.ts: [@cloudflare/wrangler]

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

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

[vite-plugin] server.ws: false leaves Vite HMR WebSocket upgrades hanging

2 participants