Skip to content

docs: remove fastify-sentry plugin#6131

Merged
gurgunday merged 1 commit intofastify:mainfrom
dnlup:patch-1
May 19, 2025
Merged

docs: remove fastify-sentry plugin#6131
gurgunday merged 1 commit intofastify:mainfrom
dnlup:patch-1

Conversation

@dnlup
Copy link
Copy Markdown
Contributor

@dnlup dnlup commented May 19, 2025 •
edited
Loading

Remove @immobiliarelabs/fastify-sentry community plugin since it's no longer maintained.

Checklist

@dnlup
docs: remove fastify-sentry plugin
06c6932 
Remove @immobiliarelabs/fastify-sentry community plugin since it's no longer maintained.

Signed-off-by: dnlup <[email protected]>
@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 19, 2025
Eomm
Eomm approved these changes May 19, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@Eomm Eomm left a comment

Choose a reason for hiding this comment

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

Your team may evaluate to deprecate the module too:

@gurgunday gurgunday merged commit 42f0f43 into fastify:main May 19, 2025
7 checks passed
@dnlup
Copy link
Copy Markdown
Contributor Author

dnlup commented May 20, 2025

Your team may evaluate to deprecate the module too:

Yes, thank you

@Uzlopak
Copy link
Copy Markdown
Contributor

Uzlopak commented May 20, 2025

@fastify/plugins

Should we transfer it to @fastify and take ownership?

@jean-michelet
Copy link
Copy Markdown
Member

jean-michelet commented May 20, 2025

There is an official integration: https://docs.sentry.io/platforms/javascript/guides/fastify/

I don't think it's necessary to maintain another wrapper.

@jsumners
Copy link
Copy Markdown
Member

jsumners commented May 20, 2025

@fastify/plugins

Should we transfer it to @fastify and take ownership?

No, we don't need to take up every decommissioned package. Particularly not ones written for specific companies's services.

@Uzlopak
Copy link
Copy Markdown
Contributor

Uzlopak commented May 20, 2025

@jsumners

This is not a valid argument, as it misses the point, that that plugin exists to fullfill a specific technical need. Anyhow in the initial post, there was no information regarding the fact that there is now a product by sentry itself, which will take care of that specific need. So it is not only maintained, but there is an alternative product.

So for me everything is cleared up.

This was referenced Aug 15, 2025
Closed
Closed
Closed
Closed
Closed
vaernion pushed a commit to Arbeidstilsynet/brevgen2 that referenced this pull request Dec 3, 2025
@vaernion
Merged PR 19273: fix(deps): update dependency fastify to v5.4.0
01cc533 
This PR contains the following updates:

| Package | Change | Age | Confidence |
|---|---|---|---|
| [fastify](https://fastify.dev/) ([source](https://github.com/fastify/fastify)) | [`5.3.3` -> `5.4.0`](https://renovatebot.com/diffs/npm/fastify/5.3.3/5.4.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/5.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/5.3.3/5.4.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

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

### [`v5.4.0`](https://github.com/fastify/fastify/releases/tag/v5.4.0)

[Compare Source](fastify/fastify@v5.3.3...v5.4.0)

#### What's Changed

- test: mv routes-\* from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6092
- test: mv skip-reply-send from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6094
- test: mv plugins from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6088
- fix(ci): ignore alternative runtime result by [@&#8203;Eomm](https://github.com/Eomm) in fastify/fastify#6125
- test: mv schema-\* from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6093
- test: mv hooks-async from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6084
- fix(types): add missing version to request.routeOptions by [@&#8203;inyourtime](https://github.com/inyourtime) in fastify/fastify#6126
- docs: remove fastify-sentry plugin by [@&#8203;dnlup](https://github.com/dnlup) in fastify/fastify#6131
- docs: add community plugins disclaimer by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6132
- docs: use cross-platform compatible info emoji by [@&#8203;Fdawgs](https://github.com/Fdawgs) in fastify/fastify#6134
- perf: nits in reply.js by [@&#8203;Cangit](https://github.com/Cangit) in fastify/fastify#6136
- docs: join core team by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6142
- docs: fix typo in hash.digest function by [@&#8203;piotr-cz](https://github.com/piotr-cz) in fastify/fastify#6145
- test: mv hooks from tap by [@&#8203;jean-michelet](https://github.com/jean-michelet) in fastify/fastify#6087
- test: improve issue 4959 unit test by [@&#8203;Uzlopak](https://github.com/Uzlopak) in fastify/fastify#6147
- chore: Bump markdownlint-cli2 from 0.17.2 to 0.18.1 by [@&#8203;dependabot](https://github.com/dependabot) in https://g...
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2026

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 Mar 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@Eomm Eomm Eomm approved these changes

@gurgunday gurgunday gurgunday approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@dnlup @Uzlopak @jean-michelet @jsumners @Eomm @gurgunday