Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
"dependencies": {
"@sentry/node": "latest || *",
"@types/node": "^18.19.1",
"fastify": "3.29.5",
"fastify": "5.8.3",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fastify v3 test app incorrectly bumped to v5

High Severity

The node-fastify-3 test application exists specifically to verify Sentry SDK compatibility with Fastify v3. Bumping fastify from 3.29.5 to 5.8.3 eliminates all Fastify v3 test coverage and makes this a near-duplicate of the already-existing node-fastify-5 test app (which uses "fastify": "^5.7.0"). A separate node-fastify-4 app also exists for v4. The application code still uses v3-specific patterns (e.g., _request.url instead of _request.routeOptions?.url, and @ts-ignore // Fastify V3 is not typed correctly comments), and calls Sentry.setupFastifyErrorHandler(app) which the v5 app doesn't use. This is a Dependabot auto-bump that doesn't account for the intentional version pinning strategy.

Fix in Cursor Fix in Web

"typescript": "~5.0.0",
"ts-node": "10.9.1"
},
Expand Down
Loading