Skip to content

fix: clone request headers instead of mutating in single-flight#2121

Merged
atilafassina merged 4 commits intomainfrom
cf-single-flight
Mar 26, 2026
Merged

fix: clone request headers instead of mutating in single-flight#2121
atilafassina merged 4 commits intomainfrom
cf-single-flight

Conversation

@atilafassina
Copy link
Member

v2 createSingleFlight headers is mutating the original request headers.
it's safer to clone the request headers like v1 does.

@atilafassina atilafassina requested a review from lxsmnsyc March 25, 2026 20:07
@netlify
Copy link

netlify bot commented Mar 25, 2026

Deploy Preview for solid-start-landing-page ready!

Name Link
🔨 Latest commit 03f4b34
🔍 Latest deploy log https://app.netlify.com/projects/solid-start-landing-page/deploys/69c53726827e9a0008f9895a
😎 Deploy Preview https://deploy-preview-2121--solid-start-landing-page.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@changeset-bot
Copy link

changeset-bot bot commented Mar 25, 2026

🦋 Changeset detected

Latest commit: 03f4b34

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

@atilafassina atilafassina changed the title Cf single flight fix: clone request headers instead of mutating in single-flight Mar 25, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 26, 2026

Open in StackBlitz

npm i https://pkg.pr.new/solidjs/solid-start/@solidjs/start@2121
npm i https://pkg.pr.new/solidjs/solid-start/@solidjs/vite-plugin-nitro-2@2121

commit: 03f4b34

@atilafassina atilafassina merged commit 2885905 into main Mar 26, 2026
10 checks passed
@atilafassina atilafassina deleted the cf-single-flight branch March 26, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants