Skip to content

Use h3/generic for runtime agnostic methods#2126

Open
huseeiin wants to merge 3 commits into
solidjs:mainfrom
huseeiin:fix-cloudflare
Open

Use h3/generic for runtime agnostic methods#2126
huseeiin wants to merge 3 commits into
solidjs:mainfrom
huseeiin:fix-cloudflare

Conversation

@huseeiin

@huseeiin huseeiin commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

now cloudflare works when not using nitro.
previously doing wrangler dev dist/server/entry-server.js (when using @cloudflare/vite-plugin, of course) the server would return a srvx NodeResponse instead of a standard Response which doesn't work.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Addresses an existing open issue: fixes #000
  • Tests for the changes have been added (for bug fixes / features)

What is the current behavior?

What is the new behavior?

Other information

@netlify

netlify Bot commented Apr 3, 2026

Copy link
Copy Markdown

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

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

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@changeset-bot

changeset-bot Bot commented Apr 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a690646

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

@pkg-pr-new

pkg-pr-new Bot commented Apr 3, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: a690646

@atilafassina atilafassina changed the title fix cloudflare Use h3/generic for runtime agnostic methods Apr 4, 2026
@brenelz

brenelz commented May 6, 2026

Copy link
Copy Markdown
Contributor

Are we good with this change?

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