Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jalmonter
Copy link
Contributor

@jalmonter jalmonter commented Dec 2, 2025

Issue: When running wrangler dev with a worker that has cron triggers, the warning message displayed an invalid URL like curl "http://localhost:undefined/cdn-cgi/handler/scheduled" because the port wasn't yet determined when the warning was logged.

Fix: Moved the warning to after the proxy server is fully ready, where the actual public URL and port are known.


  • Tests
    • Tests included/updated
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: only shows a warning
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: v3 backports no longer needed

@jalmonter jalmonter requested a review from a team as a code owner December 2, 2025 01:16
@changeset-bot
Copy link

changeset-bot bot commented Dec 2, 2025

🦋 Changeset detected

Latest commit: 0acb3cc

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

pkg-pr-new bot commented Dec 2, 2025

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

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

wrangler

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

commit: 0acb3cc

@jalmonter
Copy link
Contributor Author

This PR addresses the same issue as #11731, which was recently fixed in a separate branch by @NuroDev

@NuroDev
Copy link
Contributor

NuroDev commented Dec 23, 2025

Hey @jalmonter! Thanks for opening this PR, I didn't even realise you had already created a fix for this before opening my PR.

If you wouldn't mind adding some tests to make sure this issue doesn't crop up again (Feel free to use the tests I created in #11731) and I'll go ahead and get this merged ASAP. Or if need be I am happy to import my tests to your branch if you'd prefer? 😄

@jalmonter
Copy link
Contributor Author

Hey @NuroDev, I've imported the tests from your branch and added a changeset. This should be good to go unless checks fail, which require someone's approval before they can run.

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Jan 6, 2026
@NuroDev NuroDev merged commit ed60c4f into cloudflare:main Jan 9, 2026
40 of 41 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Jan 9, 2026
@NuroDev
Copy link
Contributor

NuroDev commented Jan 9, 2026

Merged! This will be available in the next release of wrangler.
Thank you @jalmonter 🫡

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.

2 participants