Skip to content

fix: HEAD route search#5078

Merged
mcollina merged 2 commits into
fastify:mainfrom
ivan-tymoshenko:fix-head-route-search
Oct 11, 2023
Merged

fix: HEAD route search#5078
mcollina merged 2 commits into
fastify:mainfrom
ivan-tymoshenko:fix-head-route-search

Conversation

@ivan-tymoshenko

@ivan-tymoshenko ivan-tymoshenko commented Oct 5, 2023

Copy link
Copy Markdown
Member

Fixes #4866
PR in find-my-way: delvedor/find-my-way#337

@ivan-tymoshenko ivan-tymoshenko marked this pull request as ready for review October 9, 2023 20:35
Co-authored-by: Ran Toledo <Ran.Toledo@netapp.com>
@ivan-tymoshenko

Copy link
Copy Markdown
Member Author

should be ready to merge

Comment thread lib/route.js Outdated

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Eomm Eomm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a doubt 👍🏼

}
})

test('Allow regex constraints in routes', t => {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we test the head route here?

@ivan-tymoshenko ivan-tymoshenko Oct 10, 2023

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've compied the tests from this PR #5009.
I think the main idea of them is that when you create a regular GET route, fastify creates a HEAD route automatically. Then when you create another GET route, fastify looks for a HEAD route before creating a new one. So you test HEAD routes indirectly.

@Rantoledo corerect me if I'm wrong.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is correct.

Co-authored-by: Gürgün Dayıoğlu <gurgun.dayioglu@icloud.com>
@mcollina mcollina merged commit 797f27d into fastify:main Oct 11, 2023
renovate Bot referenced this pull request in tomacheese/telcheck Oct 11, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.dev/)
([source](https://togithub.com/fastify/fastify)) | [`4.23.2` ->
`4.24.0`](https://renovatebot.com/diffs/npm/fastify/4.23.2/4.24.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.24.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.24.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.23.2/4.24.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.23.2/4.24.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify (fastify)</summary>

###
[`v4.24.0`](https://togithub.com/fastify/fastify/releases/tag/v4.24.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.23.2...v4.24.0)

#### What's Changed

- docs: Add blank line before onclose hook heading by
[@&#8203;kadoshita](https://togithub.com/kadoshita) in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- build(dependabot): ignore tap major updates by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify/pull/5047](https://togithub.com/fastify/fastify/pull/5047)
- chore: Bump actions/checkout from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/5048](https://togithub.com/fastify/fastify/pull/5048)
- chore: more perf by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5016](https://togithub.com/fastify/fastify/pull/5016)
- docs(ecosystem): add fastify-prisma by
[@&#8203;zrosenbauer](https://togithub.com/zrosenbauer) in
[https://github.com/fastify/fastify/pull/5041](https://togithub.com/fastify/fastify/pull/5041)
- test: fix ci due yup breaking by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5058](https://togithub.com/fastify/fastify/pull/5058)
- perf: optimize split params by
[@&#8203;Connormiha](https://togithub.com/Connormiha) in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- chore: implicitly infer SchemaCompiler as readonly by
[@&#8203;DemonHa](https://togithub.com/DemonHa) in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- test(logger): splitting existing tests to avoid pipeline failing for
timeout by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5064](https://togithub.com/fastify/fastify/pull/5064)
- fix(async-hooks): mixing async and callback style in preHandler option
now returns an error by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5069](https://togithub.com/fastify/fastify/pull/5069)
- fix: enhance 100 and 200 or 204 handling by
[@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- docs: add fastify-cloudflare-turnstile to ecosystem by
[@&#8203;112RG](https://togithub.com/112RG) in
[https://github.com/fastify/fastify/pull/5067](https://togithub.com/fastify/fastify/pull/5067)
- docs: Fixing a couple typos by
[@&#8203;Cadienvan](https://togithub.com/Cadienvan) in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- feat: Add `Symbol.asyncDispose` to improve DX in short lived servers.
by [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- docs: Documentation changes requested at
[#&#8203;5082](https://togithub.com/fastify/fastify/issues/5082) by
[@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5083](https://togithub.com/fastify/fastify/pull/5083)
- docs: remove thenable promisesaplus spec references by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5081](https://togithub.com/fastify/fastify/pull/5081)
- docs(ecosystem): add fastify-event-bus by
[@&#8203;Shiva127](https://togithub.com/Shiva127) in
[https://github.com/fastify/fastify/pull/5085](https://togithub.com/fastify/fastify/pull/5085)
- docs: update docs for FastifyPlugin by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5070](https://togithub.com/fastify/fastify/pull/5070)
- docs: Update for .hijack heading by
[@&#8203;jackbatzner](https://togithub.com/jackbatzner) in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)
- fix(warnings): fixed warning when accessing context property from
Request and Reply objects by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5084](https://togithub.com/fastify/fastify/pull/5084)
- fix: HEAD route search by
[@&#8203;ivan-tymoshenko](https://togithub.com/ivan-tymoshenko) in
[https://github.com/fastify/fastify/pull/5078](https://togithub.com/fastify/fastify/pull/5078)

#### New Contributors

- [@&#8203;kadoshita](https://togithub.com/kadoshita) made their first
contribution in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- [@&#8203;Connormiha](https://togithub.com/Connormiha) made their first
contribution in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- [@&#8203;DemonHa](https://togithub.com/DemonHa) made their first
contribution in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- [@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) made
their first contribution in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- [@&#8203;Cadienvan](https://togithub.com/Cadienvan) made their first
contribution in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) made
their first contribution in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- [@&#8203;jackbatzner](https://togithub.com/jackbatzner) made their
first contribution in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)

**Full Changelog**:
fastify/fastify@v4.23.2...v4.24.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/tomacheese/telcheck).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy44LjEiLCJ0YXJnZXRCcmFuY2giOiJtYXN0ZXIifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@ivan-tymoshenko ivan-tymoshenko deleted the fix-head-route-search branch October 11, 2023 16:18
jtoar referenced this pull request in redwoodjs/graphql Nov 15, 2023
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.dev/)
([source](https://togithub.com/fastify/fastify)) | [`4.23.2` ->
`4.24.3`](https://renovatebot.com/diffs/npm/fastify/4.23.2/4.24.3) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.23.2/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.23.2/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify (fastify)</summary>

###
[`v4.24.3`](https://togithub.com/fastify/fastify/releases/tag/v4.24.3)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.2...v4.24.3)

#### What's Changed

- fix: timeout on citgm tests by
[@&#8203;simone-sanfratello](https://togithub.com/simone-sanfratello) in
[https://github.com/fastify/fastify/pull/5101](https://togithub.com/fastify/fastify/pull/5101)
- chore: add missing `use strict` directives by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify/pull/5106](https://togithub.com/fastify/fastify/pull/5106)

**Full Changelog**:
fastify/fastify@v4.24.2...v4.24.3

###
[`v4.24.2`](https://togithub.com/fastify/fastify/releases/tag/v4.24.2)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.1...v4.24.2)

#### What's Changed

- fix: build problems when `Symbol.asyncDispose` type is not available.
by [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5096](https://togithub.com/fastify/fastify/pull/5096)

**Full Changelog**:
fastify/fastify@v4.24.1...v4.24.2

###
[`v4.24.1`](https://togithub.com/fastify/fastify/releases/tag/v4.24.1)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.0...v4.24.1)

#### What's Changed

- fix: citgm by
[@&#8203;simone-sanfratello](https://togithub.com/simone-sanfratello) in
[https://github.com/fastify/fastify/pull/5075](https://togithub.com/fastify/fastify/pull/5075)
- fix: HEAD route reseting by
[@&#8203;ivan-tymoshenko](https://togithub.com/ivan-tymoshenko) in
[https://github.com/fastify/fastify/pull/5090](https://togithub.com/fastify/fastify/pull/5090)

**Full Changelog**:
fastify/fastify@v4.24.0...v4.24.1

###
[`v4.24.0`](https://togithub.com/fastify/fastify/releases/tag/v4.24.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.23.2...v4.24.0)

#### What's Changed

- docs: Add blank line before onclose hook heading by
[@&#8203;kadoshita](https://togithub.com/kadoshita) in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- build(dependabot): ignore tap major updates by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify/pull/5047](https://togithub.com/fastify/fastify/pull/5047)
- chore: Bump actions/checkout from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/5048](https://togithub.com/fastify/fastify/pull/5048)
- chore: more perf by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5016](https://togithub.com/fastify/fastify/pull/5016)
- docs(ecosystem): add fastify-prisma by
[@&#8203;zrosenbauer](https://togithub.com/zrosenbauer) in
[https://github.com/fastify/fastify/pull/5041](https://togithub.com/fastify/fastify/pull/5041)
- test: fix ci due yup breaking by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5058](https://togithub.com/fastify/fastify/pull/5058)
- perf: optimize split params by
[@&#8203;Connormiha](https://togithub.com/Connormiha) in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- chore: implicitly infer SchemaCompiler as readonly by
[@&#8203;DemonHa](https://togithub.com/DemonHa) in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- test(logger): splitting existing tests to avoid pipeline failing for
timeout by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5064](https://togithub.com/fastify/fastify/pull/5064)
- fix(async-hooks): mixing async and callback style in preHandler option
now returns an error by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5069](https://togithub.com/fastify/fastify/pull/5069)
- fix: enhance 100 and 200 or 204 handling by
[@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- docs: add fastify-cloudflare-turnstile to ecosystem by
[@&#8203;112RG](https://togithub.com/112RG) in
[https://github.com/fastify/fastify/pull/5067](https://togithub.com/fastify/fastify/pull/5067)
- docs: Fixing a couple typos by
[@&#8203;Cadienvan](https://togithub.com/Cadienvan) in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- feat: Add `Symbol.asyncDispose` to improve DX in short lived servers.
by [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- docs: Documentation changes requested at
[#&#8203;5082](https://togithub.com/fastify/fastify/issues/5082) by
[@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5083](https://togithub.com/fastify/fastify/pull/5083)
- docs: remove thenable promisesaplus spec references by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5081](https://togithub.com/fastify/fastify/pull/5081)
- docs(ecosystem): add fastify-event-bus by
[@&#8203;Shiva127](https://togithub.com/Shiva127) in
[https://github.com/fastify/fastify/pull/5085](https://togithub.com/fastify/fastify/pull/5085)
- docs: update docs for FastifyPlugin by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5070](https://togithub.com/fastify/fastify/pull/5070)
- docs: Update for .hijack heading by
[@&#8203;jackbatzner](https://togithub.com/jackbatzner) in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)
- fix(warnings): fixed warning when accessing context property from
Request and Reply objects by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5084](https://togithub.com/fastify/fastify/pull/5084)
- fix: HEAD route search by
[@&#8203;ivan-tymoshenko](https://togithub.com/ivan-tymoshenko) in
[https://github.com/fastify/fastify/pull/5078](https://togithub.com/fastify/fastify/pull/5078)

#### New Contributors

- [@&#8203;kadoshita](https://togithub.com/kadoshita) made their first
contribution in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- [@&#8203;Connormiha](https://togithub.com/Connormiha) made their first
contribution in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- [@&#8203;DemonHa](https://togithub.com/DemonHa) made their first
contribution in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- [@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) made
their first contribution in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- [@&#8203;Cadienvan](https://togithub.com/Cadienvan) made their first
contribution in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) made
their first contribution in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- [@&#8203;jackbatzner](https://togithub.com/jackbatzner) made their
first contribution in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)

**Full Changelog**:
fastify/fastify@v4.23.2...v4.24.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/redwoodjs/redwood).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40Ni4wIiwidXBkYXRlZEluVmVyIjoiMzcuNDYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jtoar referenced this pull request in redwoodjs/graphql Nov 16, 2023
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.dev/)
([source](https://togithub.com/fastify/fastify)) | [`4.23.2` ->
`4.24.3`](https://renovatebot.com/diffs/npm/fastify/4.23.2/4.24.3) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.23.2/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.23.2/4.24.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

<details>
<summary>fastify/fastify (fastify)</summary>

[`v4.24.3`](https://togithub.com/fastify/fastify/releases/tag/v4.24.3)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.2...v4.24.3)

- fix: timeout on citgm tests by
[@&#8203;simone-sanfratello](https://togithub.com/simone-sanfratello) in
[https://github.com/fastify/fastify/pull/5101](https://togithub.com/fastify/fastify/pull/5101)
- chore: add missing `use strict` directives by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify/pull/5106](https://togithub.com/fastify/fastify/pull/5106)

**Full Changelog**:
fastify/fastify@v4.24.2...v4.24.3

[`v4.24.2`](https://togithub.com/fastify/fastify/releases/tag/v4.24.2)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.1...v4.24.2)

- fix: build problems when `Symbol.asyncDispose` type is not available.
by [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5096](https://togithub.com/fastify/fastify/pull/5096)

**Full Changelog**:
fastify/fastify@v4.24.1...v4.24.2

[`v4.24.1`](https://togithub.com/fastify/fastify/releases/tag/v4.24.1)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.24.0...v4.24.1)

- fix: citgm by
[@&#8203;simone-sanfratello](https://togithub.com/simone-sanfratello) in
[https://github.com/fastify/fastify/pull/5075](https://togithub.com/fastify/fastify/pull/5075)
- fix: HEAD route reseting by
[@&#8203;ivan-tymoshenko](https://togithub.com/ivan-tymoshenko) in
[https://github.com/fastify/fastify/pull/5090](https://togithub.com/fastify/fastify/pull/5090)

**Full Changelog**:
fastify/fastify@v4.24.0...v4.24.1

[`v4.24.0`](https://togithub.com/fastify/fastify/releases/tag/v4.24.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.23.2...v4.24.0)

- docs: Add blank line before onclose hook heading by
[@&#8203;kadoshita](https://togithub.com/kadoshita) in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- build(dependabot): ignore tap major updates by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify/pull/5047](https://togithub.com/fastify/fastify/pull/5047)
- chore: Bump actions/checkout from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/5048](https://togithub.com/fastify/fastify/pull/5048)
- chore: more perf by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5016](https://togithub.com/fastify/fastify/pull/5016)
- docs(ecosystem): add fastify-prisma by
[@&#8203;zrosenbauer](https://togithub.com/zrosenbauer) in
[https://github.com/fastify/fastify/pull/5041](https://togithub.com/fastify/fastify/pull/5041)
- test: fix ci due yup breaking by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/5058](https://togithub.com/fastify/fastify/pull/5058)
- perf: optimize split params by
[@&#8203;Connormiha](https://togithub.com/Connormiha) in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- chore: implicitly infer SchemaCompiler as readonly by
[@&#8203;DemonHa](https://togithub.com/DemonHa) in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- test(logger): splitting existing tests to avoid pipeline failing for
timeout by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5064](https://togithub.com/fastify/fastify/pull/5064)
- fix(async-hooks): mixing async and callback style in preHandler option
now returns an error by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5069](https://togithub.com/fastify/fastify/pull/5069)
- fix: enhance 100 and 200 or 204 handling by
[@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- docs: add fastify-cloudflare-turnstile to ecosystem by
[@&#8203;112RG](https://togithub.com/112RG) in
[https://github.com/fastify/fastify/pull/5067](https://togithub.com/fastify/fastify/pull/5067)
- docs: Fixing a couple typos by
[@&#8203;Cadienvan](https://togithub.com/Cadienvan) in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- feat: Add `Symbol.asyncDispose` to improve DX in short lived servers.
by [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- docs: Documentation changes requested at
[#&#8203;5082](https://togithub.com/fastify/fastify/issues/5082) by
[@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) in
[https://github.com/fastify/fastify/pull/5083](https://togithub.com/fastify/fastify/pull/5083)
- docs: remove thenable promisesaplus spec references by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5081](https://togithub.com/fastify/fastify/pull/5081)
- docs(ecosystem): add fastify-event-bus by
[@&#8203;Shiva127](https://togithub.com/Shiva127) in
[https://github.com/fastify/fastify/pull/5085](https://togithub.com/fastify/fastify/pull/5085)
- docs: update docs for FastifyPlugin by
[@&#8203;dancastillo](https://togithub.com/dancastillo) in
[https://github.com/fastify/fastify/pull/5070](https://togithub.com/fastify/fastify/pull/5070)
- docs: Update for .hijack heading by
[@&#8203;jackbatzner](https://togithub.com/jackbatzner) in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)
- fix(warnings): fixed warning when accessing context property from
Request and Reply objects by
[@&#8203;giuliowaitforitdavide](https://togithub.com/giuliowaitforitdavide)
in
[https://github.com/fastify/fastify/pull/5084](https://togithub.com/fastify/fastify/pull/5084)
- fix: HEAD route search by
[@&#8203;ivan-tymoshenko](https://togithub.com/ivan-tymoshenko) in
[https://github.com/fastify/fastify/pull/5078](https://togithub.com/fastify/fastify/pull/5078)

- [@&#8203;kadoshita](https://togithub.com/kadoshita) made their first
contribution in
[https://github.com/fastify/fastify/pull/5042](https://togithub.com/fastify/fastify/pull/5042)
- [@&#8203;Connormiha](https://togithub.com/Connormiha) made their first
contribution in
[https://github.com/fastify/fastify/pull/5057](https://togithub.com/fastify/fastify/pull/5057)
- [@&#8203;DemonHa](https://togithub.com/DemonHa) made their first
contribution in
[https://github.com/fastify/fastify/pull/5060](https://togithub.com/fastify/fastify/pull/5060)
- [@&#8203;Iamshankhadeep](https://togithub.com/Iamshankhadeep) made
their first contribution in
[https://github.com/fastify/fastify/pull/5056](https://togithub.com/fastify/fastify/pull/5056)
- [@&#8203;Cadienvan](https://togithub.com/Cadienvan) made their first
contribution in
[https://github.com/fastify/fastify/pull/5079](https://togithub.com/fastify/fastify/pull/5079)
- [@&#8203;arthurfiorette](https://togithub.com/arthurfiorette) made
their first contribution in
[https://github.com/fastify/fastify/pull/5082](https://togithub.com/fastify/fastify/pull/5082)
- [@&#8203;jackbatzner](https://togithub.com/jackbatzner) made their
first contribution in
[https://github.com/fastify/fastify/pull/5088](https://togithub.com/fastify/fastify/pull/5088)

**Full Changelog**:
fastify/fastify@v4.23.2...v4.24.0

</details>

---

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/redwoodjs/redwood).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40Ni4wIiwidXBkYXRlZEluVmVyIjoiMzcuNDYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HEAD requests are resolving to wildcard routes instead of named routes

6 participants