Skip to content

Upgrade to router.fetch#103

Open
Cyb3r-Jak3 wants to merge 1 commit into
cloudflare:mainfrom
Cyb3r-Jak3:router-fetch
Open

Upgrade to router.fetch#103
Cyb3r-Jak3 wants to merge 1 commit into
cloudflare:mainfrom
Cyb3r-Jak3:router-fetch

Conversation

@Cyb3r-Jak3

Copy link
Copy Markdown

Swap router.handle to router.fetch. router.handle is being dropped in itty-router@v5

https://itty.dev/itty-router/migrations/v4-v5#_1-router-handle-is-deprecated-in-favor-of-router-fetch

@harryzcy

harryzcy commented May 2, 2025

Copy link
Copy Markdown
Contributor

At some point in v4, we began to support (at the cost of extra bytes) boththe original router.handle and the newer router.fetch

Do you know from what version they started supporting that? Probably good to bump minor version of itty-router to make sure it's supported

@Cyb3r-Jak3

Copy link
Copy Markdown
Author

Looks like v4.0.27 based of this commit, kwhitley/itty-router@53da003.

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