Skip to content

fix: do not double send the response if the request is destroyed but not aborted#4963

Merged
Uzlopak merged 1 commit intomainfrom
fix-4959
Aug 9, 2023
Merged

fix: do not double send the response if the request is destroyed but not aborted#4963
Uzlopak merged 1 commit intomainfrom
fix-4959

Conversation

@mcollina
Copy link
Member

@mcollina mcollina commented Aug 9, 2023

Fixes #4959

Checklist

…borted

Signed-off-by: Matteo Collina <hello@matteocollina.com>
@mcollina mcollina requested a review from climba03003 August 9, 2023 07:19
@mcollina mcollina changed the title Do not double send the response if the request is destroyed but not a… fix: do not double send the response if the request is destroyed but not aborted Aug 9, 2023
@Uzlopak Uzlopak added the benchmark Label to run benchmark against PR and main branch label Aug 9, 2023
@github-actions
Copy link

github-actions bot commented Aug 9, 2023

Node: 16
PR: [1] 1010k requests in 30.1s, 190 MB read
MAIN: [1] 983k requests in 30.07s, 185 MB read


Node: 18
PR: [1] 1494k requests in 30.05s, 281 MB read
MAIN: [1] 1477k requests in 30.05s, 278 MB read


Node: 20
PR: [1] 1513k requests in 30.05s, 284 MB read
MAIN: [1] 1521k requests in 30.05s, 286 MB read

@github-actions github-actions bot removed the benchmark Label to run benchmark against PR and main branch label Aug 9, 2023
@Uzlopak Uzlopak merged commit c235d2d into main Aug 9, 2023
@Uzlopak Uzlopak deleted the fix-4959 branch August 9, 2023 08:50
renovate bot referenced this pull request in tomacheese/telcheck Aug 27, 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.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.21.0` ->
`4.22.0`](https://renovatebot.com/diffs/npm/fastify/4.21.0/4.22.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

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

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

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.21.0...v4.22.0)

#### What's Changed

- make FastifySchemaValidationError.params wider by
[@&#8203;cm-ayf](https://togithub.com/cm-ayf) in
[https://github.com/fastify/fastify/pull/4476](https://togithub.com/fastify/fastify/pull/4476)
- docs(ecosystem): add fastify-hashids by
[@&#8203;andersonjoseph](https://togithub.com/andersonjoseph) in
[https://github.com/fastify/fastify/pull/4934](https://togithub.com/fastify/fastify/pull/4934)
- fix: hasPlugin does not track parent plugins by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4929](https://togithub.com/fastify/fastify/pull/4929)
- docs: early hints plugin is fastify plugin by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4947](https://togithub.com/fastify/fastify/pull/4947)
- chore: add pull request title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4951](https://togithub.com/fastify/fastify/pull/4951)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.29.6 to 0.30.2 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4952](https://togithub.com/fastify/fastify/pull/4952)
- ci: improve pr title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4953](https://togithub.com/fastify/fastify/pull/4953)
- ci: fix warnings in benchmark workflows by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4954](https://togithub.com/fastify/fastify/pull/4954)
- docs: fix removeAdditional comment by
[@&#8203;G0maa](https://togithub.com/G0maa) in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- fix: Try to fix parser benchmark workflow by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[https://github.com/fastify/fastify/pull/4956](https://togithub.com/fastify/fastify/pull/4956)
- fix: infer correct hook handler by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4945](https://togithub.com/fastify/fastify/pull/4945)
- fix: do not double send the response if the request is destroyed but
not aborted by [@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4963](https://togithub.com/fastify/fastify/pull/4963)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.30.4 to 0.31.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4973](https://togithub.com/fastify/fastify/pull/4973)
- fix: bodyLimit must be applied on fully decoded body by
[@&#8203;sergburn](https://togithub.com/sergburn) in
[https://github.com/fastify/fastify/pull/4969](https://togithub.com/fastify/fastify/pull/4969)
- chore: updates
[@&#8203;typescript-eslint/eslint-plugin](https://togithub.com/typescript-eslint/eslint-plugin)
and
[@&#8203;typescript-eslint/parser](https://togithub.com/typescript-eslint/parser)
by [@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4977](https://togithub.com/fastify/fastify/pull/4977)
- chore: use group dependencies in dependabot by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4979](https://togithub.com/fastify/fastify/pull/4979)
- chore: fix ci bench by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4983](https://togithub.com/fastify/fastify/pull/4983)
- fix: require.cache is undefined breaks SEA by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4982](https://togithub.com/fastify/fastify/pull/4982)
- ci: remove sync next wf by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4985](https://togithub.com/fastify/fastify/pull/4985)
- docs: remove mixing ES6 and commomjs in the example by
[@&#8203;thenicolau](https://togithub.com/thenicolau) in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- fix: errorHandler callback should utilize TypeProvider by
[@&#8203;muan](https://togithub.com/muan) in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- types: add onRoute to ApplicationHookLookup by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4968](https://togithub.com/fastify/fastify/pull/4968)
- chore: make tests pass on ipv4 only machine by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4997](https://togithub.com/fastify/fastify/pull/4997)
- fix: Set `FastifyRequest.id` type as a string by
[@&#8203;samchungy](https://togithub.com/samchungy) in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

#### New Contributors

- [@&#8203;G0maa](https://togithub.com/G0maa) made their first
contribution in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- [@&#8203;thenicolau](https://togithub.com/thenicolau) made their first
contribution in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- [@&#8203;muan](https://togithub.com/muan) made their first
contribution in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- [@&#8203;samchungy](https://togithub.com/samchungy) made their first
contribution in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

**Full Changelog**:
fastify/fastify@v4.21.0...v4.22.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:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ni4wIiwidXBkYXRlZEluVmVyIjoiMzYuNTYuMCIsInRhcmdldEJyYW5jaCI6Im1hc3RlciJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate bot referenced this pull request in redwoodjs/graphql Aug 30, 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.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.21.0` ->
`4.22.0`](https://renovatebot.com/diffs/npm/fastify/4.21.0/4.22.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

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

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

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.21.0...v4.22.0)

##### What's Changed

- make FastifySchemaValidationError.params wider by
[@&#8203;cm-ayf](https://togithub.com/cm-ayf) in
[https://github.com/fastify/fastify/pull/4476](https://togithub.com/fastify/fastify/pull/4476)
- docs(ecosystem): add fastify-hashids by
[@&#8203;andersonjoseph](https://togithub.com/andersonjoseph) in
[https://github.com/fastify/fastify/pull/4934](https://togithub.com/fastify/fastify/pull/4934)
- fix: hasPlugin does not track parent plugins by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4929](https://togithub.com/fastify/fastify/pull/4929)
- docs: early hints plugin is fastify plugin by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4947](https://togithub.com/fastify/fastify/pull/4947)
- chore: add pull request title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4951](https://togithub.com/fastify/fastify/pull/4951)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.29.6 to 0.30.2 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4952](https://togithub.com/fastify/fastify/pull/4952)
- ci: improve pr title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4953](https://togithub.com/fastify/fastify/pull/4953)
- ci: fix warnings in benchmark workflows by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4954](https://togithub.com/fastify/fastify/pull/4954)
- docs: fix removeAdditional comment by
[@&#8203;G0maa](https://togithub.com/G0maa) in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- fix: Try to fix parser benchmark workflow by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[https://github.com/fastify/fastify/pull/4956](https://togithub.com/fastify/fastify/pull/4956)
- fix: infer correct hook handler by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4945](https://togithub.com/fastify/fastify/pull/4945)
- fix: do not double send the response if the request is destroyed but
not aborted by [@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4963](https://togithub.com/fastify/fastify/pull/4963)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.30.4 to 0.31.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4973](https://togithub.com/fastify/fastify/pull/4973)
- fix: bodyLimit must be applied on fully decoded body by
[@&#8203;sergburn](https://togithub.com/sergburn) in
[https://github.com/fastify/fastify/pull/4969](https://togithub.com/fastify/fastify/pull/4969)
- chore: updates
[@&#8203;typescript-eslint/eslint-plugin](https://togithub.com/typescript-eslint/eslint-plugin)
and
[@&#8203;typescript-eslint/parser](https://togithub.com/typescript-eslint/parser)
by [@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4977](https://togithub.com/fastify/fastify/pull/4977)
- chore: use group dependencies in dependabot by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4979](https://togithub.com/fastify/fastify/pull/4979)
- chore: fix ci bench by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4983](https://togithub.com/fastify/fastify/pull/4983)
- fix: require.cache is undefined breaks SEA by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4982](https://togithub.com/fastify/fastify/pull/4982)
- ci: remove sync next wf by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4985](https://togithub.com/fastify/fastify/pull/4985)
- docs: remove mixing ES6 and commomjs in the example by
[@&#8203;thenicolau](https://togithub.com/thenicolau) in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- fix: errorHandler callback should utilize TypeProvider by
[@&#8203;muan](https://togithub.com/muan) in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- types: add onRoute to ApplicationHookLookup by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4968](https://togithub.com/fastify/fastify/pull/4968)
- chore: make tests pass on ipv4 only machine by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4997](https://togithub.com/fastify/fastify/pull/4997)
- fix: Set `FastifyRequest.id` type as a string by
[@&#8203;samchungy](https://togithub.com/samchungy) in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

##### New Contributors

- [@&#8203;G0maa](https://togithub.com/G0maa) made their first
contribution in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- [@&#8203;thenicolau](https://togithub.com/thenicolau) made their first
contribution in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- [@&#8203;muan](https://togithub.com/muan) made their first
contribution in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- [@&#8203;samchungy](https://togithub.com/samchungy) made their first
contribution in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

**Full Changelog**:
fastify/fastify@v4.21.0...v4.22.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:eyJjcmVhdGVkSW5WZXIiOiIzNi42OC4xIiwidXBkYXRlZEluVmVyIjoiMzYuNjguMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
mcollina added a commit that referenced this pull request Aug 31, 2023
jtoar referenced this pull request in redwoodjs/graphql Sep 2, 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.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.21.0` ->
`4.22.0`](https://renovatebot.com/diffs/npm/fastify/4.21.0/4.22.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.21.0/4.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

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

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

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.21.0...v4.22.0)

##### What's Changed

- make FastifySchemaValidationError.params wider by
[@&#8203;cm-ayf](https://togithub.com/cm-ayf) in
[https://github.com/fastify/fastify/pull/4476](https://togithub.com/fastify/fastify/pull/4476)
- docs(ecosystem): add fastify-hashids by
[@&#8203;andersonjoseph](https://togithub.com/andersonjoseph) in
[https://github.com/fastify/fastify/pull/4934](https://togithub.com/fastify/fastify/pull/4934)
- fix: hasPlugin does not track parent plugins by
[@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4929](https://togithub.com/fastify/fastify/pull/4929)
- docs: early hints plugin is fastify plugin by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4947](https://togithub.com/fastify/fastify/pull/4947)
- chore: add pull request title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4951](https://togithub.com/fastify/fastify/pull/4951)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.29.6 to 0.30.2 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4952](https://togithub.com/fastify/fastify/pull/4952)
- ci: improve pr title check by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4953](https://togithub.com/fastify/fastify/pull/4953)
- ci: fix warnings in benchmark workflows by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4954](https://togithub.com/fastify/fastify/pull/4954)
- docs: fix removeAdditional comment by
[@&#8203;G0maa](https://togithub.com/G0maa) in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- fix: Try to fix parser benchmark workflow by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[https://github.com/fastify/fastify/pull/4956](https://togithub.com/fastify/fastify/pull/4956)
- fix: infer correct hook handler by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4945](https://togithub.com/fastify/fastify/pull/4945)
- fix: do not double send the response if the request is destroyed but
not aborted by [@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4963](https://togithub.com/fastify/fastify/pull/4963)
- chore: Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.30.4 to 0.31.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify/pull/4973](https://togithub.com/fastify/fastify/pull/4973)
- fix: bodyLimit must be applied on fully decoded body by
[@&#8203;sergburn](https://togithub.com/sergburn) in
[https://github.com/fastify/fastify/pull/4969](https://togithub.com/fastify/fastify/pull/4969)
- chore: updates
[@&#8203;typescript-eslint/eslint-plugin](https://togithub.com/typescript-eslint/eslint-plugin)
and
[@&#8203;typescript-eslint/parser](https://togithub.com/typescript-eslint/parser)
by [@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4977](https://togithub.com/fastify/fastify/pull/4977)
- chore: use group dependencies in dependabot by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4979](https://togithub.com/fastify/fastify/pull/4979)
- chore: fix ci bench by [@&#8203;Eomm](https://togithub.com/Eomm) in
[https://github.com/fastify/fastify/pull/4983](https://togithub.com/fastify/fastify/pull/4983)
- fix: require.cache is undefined breaks SEA by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify/pull/4982](https://togithub.com/fastify/fastify/pull/4982)
- ci: remove sync next wf by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4985](https://togithub.com/fastify/fastify/pull/4985)
- docs: remove mixing ES6 and commomjs in the example by
[@&#8203;thenicolau](https://togithub.com/thenicolau) in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- fix: errorHandler callback should utilize TypeProvider by
[@&#8203;muan](https://togithub.com/muan) in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- types: add onRoute to ApplicationHookLookup by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[https://github.com/fastify/fastify/pull/4968](https://togithub.com/fastify/fastify/pull/4968)
- chore: make tests pass on ipv4 only machine by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify/pull/4997](https://togithub.com/fastify/fastify/pull/4997)
- fix: Set `FastifyRequest.id` type as a string by
[@&#8203;samchungy](https://togithub.com/samchungy) in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

##### New Contributors

- [@&#8203;G0maa](https://togithub.com/G0maa) made their first
contribution in
[https://github.com/fastify/fastify/pull/4948](https://togithub.com/fastify/fastify/pull/4948)
- [@&#8203;thenicolau](https://togithub.com/thenicolau) made their first
contribution in
[https://github.com/fastify/fastify/pull/4990](https://togithub.com/fastify/fastify/pull/4990)
- [@&#8203;muan](https://togithub.com/muan) made their first
contribution in
[https://github.com/fastify/fastify/pull/4989](https://togithub.com/fastify/fastify/pull/4989)
- [@&#8203;samchungy](https://togithub.com/samchungy) made their first
contribution in
[https://github.com/fastify/fastify/pull/4992](https://togithub.com/fastify/fastify/pull/4992)

**Full Changelog**:
fastify/fastify@v4.21.0...v4.22.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:eyJjcmVhdGVkSW5WZXIiOiIzNi42OC4xIiwidXBkYXRlZEluVmVyIjoiMzYuNjguMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

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

github-actions bot commented Aug 9, 2024

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 Aug 9, 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.

onSend hook can be called twice

3 participants