Skip to content

Fix request route config deprecation warnings#409

Merged
gurgunday merged 4 commits into
fastify:masterfrom
TTPO100AJIEX:master
Sep 13, 2023
Merged

Fix request route config deprecation warnings#409
gurgunday merged 4 commits into
fastify:masterfrom
TTPO100AJIEX:master

Conversation

@TTPO100AJIEX

@TTPO100AJIEX TTPO100AJIEX commented Sep 13, 2023

Copy link
Copy Markdown
Contributor

request.routeConfig has been deprecated in Fastify v4.23.0, which causes warnings in fastify-static.

This PR replaces the deprecated api with the recommended request.routeOptions.config property.

Checklist

@Uzlopak

Uzlopak commented Sep 13, 2023

Copy link
Copy Markdown
Contributor

Doing it like this could be problematic, because if somebody uses an older fastify version it could break.

Please add a version constraint

https://github.com/fastify/fastify-plugin#fastify-version

@TTPO100AJIEX

TTPO100AJIEX commented Sep 13, 2023

Copy link
Copy Markdown
Contributor Author

Done

@Uzlopak

Uzlopak commented Sep 13, 2023

Copy link
Copy Markdown
Contributor

Now lets run the cicd pipeline. ;)

@Eomm
@climba03003
@mcollina

We should be careful when we adapt the changes.

climba03003
climba03003 requested changes Sep 13, 2023
View reviewed changes
Comment thread index.js Outdated
@TTPO100AJIEX @climba03003
Update version requirement
5b33271 
Co-authored-by: KaKa <23028015+climba03003@users.noreply.github.com>
gurgunday
gurgunday requested changes Sep 13, 2023
View reviewed changes

@gurgunday gurgunday left a comment •
edited
Loading

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.

What? Isn't this the same as 4.23.x? Shouldn't it be ^4.23.0?

Edit: moved the convo up

@climba03003

climba03003 commented Sep 13, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

What? Isn't this the same as 4.23.x? Shouldn't it be ^4.23.0?

Oops, Yes. It should be ^ @TTPO100AJIEX

@TTPO100AJIEX

TTPO100AJIEX commented Sep 13, 2023

Copy link
Copy Markdown
Contributor Author

Yes, already fixed that

gurgunday
gurgunday approved these changes Sep 13, 2023
View reviewed changes

@gurgunday gurgunday 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

@gurgunday gurgunday merged commit e0a816e into fastify:master Sep 13, 2023
@Uzlopak

Uzlopak commented Sep 13, 2023

Copy link
Copy Markdown
Contributor

Thank you for your contribution.

renovate Bot referenced this pull request in redwoodjs/graphql Sep 16, 2023
@renovate
fix(deps): update dependency @fastify/static to v6.11.1 (#9182)
552eec3 
[![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/static](https://togithub.com/fastify/fastify-static) |
[`6.11.0` ->
`6.11.1`](https://renovatebot.com/diffs/npm/@fastify%2fstatic/6.11.0/6.11.1)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@fastify%2fstatic/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@fastify%2fstatic/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@fastify%2fstatic/6.11.0/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@fastify%2fstatic/6.11.0/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify-static (@&#8203;fastify/static)</summary>

###
[`v6.11.1`](https://togithub.com/fastify/fastify-static/releases/tag/v6.11.1)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.11.0...v6.11.1)

##### What's Changed

- Remove test/content-type/sample.jpg.br by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify-static/pull/400](https://togithub.com/fastify/fastify-static/pull/400)
- build(deps-dev): bump tsd from 0.28.1 to 0.29.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify-static/pull/401](https://togithub.com/fastify/fastify-static/pull/401)
- added some small code improvements by
[@&#8203;turnerran](https://togithub.com/turnerran) in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- (more) small performance/clarity improvements by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/405](https://togithub.com/fastify/fastify-static/pull/405)
- perf: use `node:` prefix to bypass require.cache call for builtins by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/407](https://togithub.com/fastify/fastify-static/pull/407)
- use native stream by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/408](https://togithub.com/fastify/fastify-static/pull/408)
- Fix request route config deprecation warnings by
[@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

##### New Contributors

- [@&#8203;turnerran](https://togithub.com/turnerran) made their first
contribution in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- [@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) made their
first contribution in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

**Full Changelog**:
fastify/fastify-static@v6.11.0...v6.11.1

</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:eyJjcmVhdGVkSW5WZXIiOiIzNi44My4wIiwidXBkYXRlZEluVmVyIjoiMzYuODMuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jtoar referenced this pull request in redwoodjs/graphql Sep 16, 2023
@renovate @jtoar
fix(deps): update dependency @fastify/static to v6.11.1 (#9182)
68f8ddc 
[![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/static](https://togithub.com/fastify/fastify-static) |
[`6.11.0` ->
`6.11.1`](https://renovatebot.com/diffs/npm/@fastify%2fstatic/6.11.0/6.11.1)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@fastify%2fstatic/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@fastify%2fstatic/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@fastify%2fstatic/6.11.0/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@fastify%2fstatic/6.11.0/6.11.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify-static (@&#8203;fastify/static)</summary>

###
[`v6.11.1`](https://togithub.com/fastify/fastify-static/releases/tag/v6.11.1)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.11.0...v6.11.1)

##### What's Changed

- Remove test/content-type/sample.jpg.br by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify-static/pull/400](https://togithub.com/fastify/fastify-static/pull/400)
- build(deps-dev): bump tsd from 0.28.1 to 0.29.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify-static/pull/401](https://togithub.com/fastify/fastify-static/pull/401)
- added some small code improvements by
[@&#8203;turnerran](https://togithub.com/turnerran) in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- (more) small performance/clarity improvements by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/405](https://togithub.com/fastify/fastify-static/pull/405)
- perf: use `node:` prefix to bypass require.cache call for builtins by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/407](https://togithub.com/fastify/fastify-static/pull/407)
- use native stream by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/408](https://togithub.com/fastify/fastify-static/pull/408)
- Fix request route config deprecation warnings by
[@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

##### New Contributors

- [@&#8203;turnerran](https://togithub.com/turnerran) made their first
contribution in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- [@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) made their
first contribution in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

**Full Changelog**:
fastify/fastify-static@v6.11.0...v6.11.1

</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:eyJjcmVhdGVkSW5WZXIiOiIzNi44My4wIiwidXBkYXRlZEluVmVyIjoiMzYuODMuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate Bot referenced this pull request in specfy/specfy Dec 4, 2023
@renovate
fix(deps): update dependency @fastify/static to v6.12.0 (#393)
dbb8a5c 
[![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/static](https://togithub.com/fastify/fastify-static) |
[`6.10.2` ->
`6.12.0`](https://renovatebot.com/diffs/npm/@fastify%2fstatic/6.10.2/6.12.0)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/@fastify%2fstatic/6.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@fastify%2fstatic/6.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@fastify%2fstatic/6.10.2/6.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@fastify%2fstatic/6.10.2/6.12.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify-static (@&#8203;fastify/static)</summary>

###
[`v6.12.0`](https://togithub.com/fastify/fastify-static/releases/tag/v6.12.0)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.11.2...v6.12.0)

#### What's Changed

- fix eslint by [@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/415](https://togithub.com/fastify/fastify-static/pull/415)
- refactor: add `u` unicode flag to regex by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/418](https://togithub.com/fastify/fastify-static/pull/418)
- refactor: remove unused bindings by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/417](https://togithub.com/fastify/fastify-static/pull/417)
- perf(index): check set size over converting to array to check length
by [@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/416](https://togithub.com/fastify/fastify-static/pull/416)
- chore(package): explicitly declare js module type by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/419](https://togithub.com/fastify/fastify-static/pull/419)
- test: replace `typeof` undefined check by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/420](https://togithub.com/fastify/fastify-static/pull/420)

**Full Changelog**:
fastify/fastify-static@v6.11.2...v6.12.0

###
[`v6.11.2`](https://togithub.com/fastify/fastify-static/releases/tag/v6.11.2)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.11.1...v6.11.2)

#### What's Changed

- fix: allow fallback to fastify@2.22 route config by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[https://github.com/fastify/fastify-static/pull/410](https://togithub.com/fastify/fastify-static/pull/410)

**Full Changelog**:
fastify/fastify-static@v6.11.1...v6.11.2

###
[`v6.11.1`](https://togithub.com/fastify/fastify-static/releases/tag/v6.11.1)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.11.0...v6.11.1)

#### What's Changed

- Remove test/content-type/sample.jpg.br by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/fastify/fastify-static/pull/400](https://togithub.com/fastify/fastify-static/pull/400)
- build(deps-dev): bump tsd from 0.28.1 to 0.29.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify-static/pull/401](https://togithub.com/fastify/fastify-static/pull/401)
- added some small code improvements by
[@&#8203;turnerran](https://togithub.com/turnerran) in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- (more) small performance/clarity improvements by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/405](https://togithub.com/fastify/fastify-static/pull/405)
- perf: use `node:` prefix to bypass require.cache call for builtins by
[@&#8203;Fdawgs](https://togithub.com/Fdawgs) in
[https://github.com/fastify/fastify-static/pull/407](https://togithub.com/fastify/fastify-static/pull/407)
- use native stream by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/408](https://togithub.com/fastify/fastify-static/pull/408)
- Fix request route config deprecation warnings by
[@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

#### New Contributors

- [@&#8203;turnerran](https://togithub.com/turnerran) made their first
contribution in
[https://github.com/fastify/fastify-static/pull/399](https://togithub.com/fastify/fastify-static/pull/399)
- [@&#8203;TTPO100AJIEX](https://togithub.com/TTPO100AJIEX) made their
first contribution in
[https://github.com/fastify/fastify-static/pull/409](https://togithub.com/fastify/fastify-static/pull/409)

**Full Changelog**:
fastify/fastify-static@v6.11.0...v6.11.1

###
[`v6.11.0`](https://togithub.com/fastify/fastify-static/releases/tag/v6.11.0)

[Compare
Source](https://togithub.com/fastify/fastify-static/compare/v6.10.2...v6.11.0)

#### What's Changed

- Update README with 404+wildcard behavior by
[@&#8203;Ethan-Arrowood](https://togithub.com/Ethan-Arrowood) in
[https://github.com/fastify/fastify-static/pull/390](https://togithub.com/fastify/fastify-static/pull/390)
- add tests for
[#&#8203;234](https://togithub.com/fastify/fastify-static/issues/234) by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/391](https://togithub.com/fastify/fastify-static/pull/391)
- remove linting from the test script by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/394](https://togithub.com/fastify/fastify-static/pull/394)
- Remove redundant parameter by
[@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/395](https://togithub.com/fastify/fastify-static/pull/395)
- build(deps-dev): bump
[@&#8203;typescript-eslint/eslint-plugin](https://togithub.com/typescript-eslint/eslint-plugin)
from 2.34.0 to 6.2.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/fastify/fastify-static/pull/397](https://togithub.com/fastify/fastify-static/pull/397)
- add url type by [@&#8203;gurgunday](https://togithub.com/gurgunday) in
[https://github.com/fastify/fastify-static/pull/398](https://togithub.com/fastify/fastify-static/pull/398)

#### New Contributors

- [@&#8203;Ethan-Arrowood](https://togithub.com/Ethan-Arrowood) made
their first contribution in
[https://github.com/fastify/fastify-static/pull/390](https://togithub.com/fastify/fastify-static/pull/390)
- [@&#8203;gurgunday](https://togithub.com/gurgunday) made their first
contribution in
[https://github.com/fastify/fastify-static/pull/391](https://togithub.com/fastify/fastify-static/pull/391)

**Full Changelog**:
fastify/fastify-static@v6.10.2...v6.11.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 4pm every weekday" in timezone
Europe/Paris, 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/specfy/specfy).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuODEuMyIsInRhcmdldEJyYW5jaCI6ImNob3JlL3Jlbm92YXRlQmFzZUJyYW5jaCJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@climba03003 climba03003 climba03003 approved these changes

@gurgunday gurgunday gurgunday approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@TTPO100AJIEX @Uzlopak @climba03003 @gurgunday