feat(deps)!: bump http-proxy-middleware to 4.0.0-beta.3#200
Conversation
There was a problem hiding this comment.
Pull request overview
Upgrades the dev/test dependency http-proxy-middleware to 4.0.0-beta.3 and adjusts the IPv6 “allowed-hosts” websocket E2E proxy setup to use a v4-compatible configuration while preserving the intent of the test.
Changes:
- Bump
http-proxy-middlewareto4.0.0-beta.3. - Update the IPv6 allowed-hosts websocket E2E test proxy to use an object
targetand explicitly restore theHostheader for HTTP/WS. - Refresh
pnpm-lock.yamlto reflect the new dependency graph and engine constraints.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/e2e/allowed-hosts.test.js | Adjust IPv6 proxy setup to avoid v4 IPv6 string-target issues and keep WS test coverage. |
| package.json | Pin http-proxy-middleware to 4.0.0-beta.3. |
| pnpm-lock.yaml | Lockfile updates for v4 beta and its transitive deps/engines. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 656d6819eb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
http-proxy-middlewareto4.0.0-beta.3allowed-hostswebsocket test exercisinghttp-proxy-middlewareby switching that proxy setup to the minimal v4-compatible configurationRelated Links