Skip to content

SQC-810 Fix tls configuration for hyperdrive local dev#13314

Merged
jamesopstad merged 1 commit intocloudflare:mainfrom
Ltadrian:agracia/SQC-810-fix-hyperdrive-local-dev
Apr 7, 2026
Merged

SQC-810 Fix tls configuration for hyperdrive local dev#13314
jamesopstad merged 1 commit intocloudflare:mainfrom
Ltadrian:agracia/SQC-810-fix-hyperdrive-local-dev

Conversation

@Ltadrian
Copy link
Copy Markdown
Contributor

@Ltadrian Ltadrian commented Apr 6, 2026

Fixes failed tls connections due to strict self signed cert validation

Fixes
https://x.com/Checkm3out/status/2040080426518093944

Describe your change...
Disable verifying self signed certs when using npx wrangler dev
Matches psql experience where this validation is disabled by default

  • Tests
    • Tests included/updated - existing tests should still suffice as this is just a small configuration change
    • Automated tests not possible - manual testing has been completed as follows:
  1. Created digital ocean database cluster similar to customer environment
  2. ran pnpm run build --filter wrangler to build wrangler locally
  3. set up worker with hyperdrive binding and set digital ocean connection string on localConnectionString
  4. run node ~/workers-sdk/packages/wrangler/wrangler-dist/cli.js dev
  5. connected to digital ocean database as expected
    without the change it fails with 500 error

Open with Devin

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 6, 2026

🦋 Changeset detected

Latest commit: 560c72d

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
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 6, 2026

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: 560c72d

- fixes failed tls connections due to strict self signed cert validation
@Ltadrian Ltadrian force-pushed the agracia/SQC-810-fix-hyperdrive-local-dev branch from ac643c5 to 560c72d Compare April 6, 2026 17:01
@Ltadrian Ltadrian marked this pull request as ready for review April 6, 2026 17:06
@workers-devprod workers-devprod requested review from a team and dario-piotrowicz and removed request for a team April 6, 2026 17:06
@workers-devprod
Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • packages/miniflare/src/plugins/hyperdrive/hyperdrive-proxy.ts: [@cloudflare/wrangler]

Copy link
Copy Markdown
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

@knickish
Copy link
Copy Markdown
Contributor

knickish commented Apr 7, 2026

LGTM for hyperdrive

@jamesopstad jamesopstad removed the request for review from dario-piotrowicz April 7, 2026 14:58
@jamesopstad jamesopstad merged commit 7a60d4b into cloudflare:main Apr 7, 2026
54 checks passed
@github-project-automation github-project-automation bot moved this from Untriaged to Done in workers-sdk Apr 7, 2026
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.

5 participants