Skip to content

feat: update to avoid brace-extansion issue#20205

Closed
MaitreGEEK wants to merge 1 commit intogetsentry:masterfrom
MaitreGEEK:patch-1
Closed

feat: update to avoid brace-extansion issue#20205
MaitreGEEK wants to merge 1 commit intogetsentry:masterfrom
MaitreGEEK:patch-1

Conversation

@MaitreGEEK
Copy link
Copy Markdown

@sentry/node › @fastify/otel › minimatch › brace-expansion
moderate: brace-expansion: Zero-step sequence causes process hang and memory exhaustion - GHSA-f886-m6hf-6m8v

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • [x ] If you've added code that should be tested, please add tests.
  • [ x] Ensure your code lints and the test suite passes (yarn lint) & (yarn test).
  • [x ] Link an issue if there is one related to your pull request. If no issue is linked, one will be auto-generated and linked.

Closes #20204 #20204

@sentry/node › @fastify/otel › minimatch › brace-expansion
  moderate: brace-expansion: Zero-step sequence causes process hang and memory exhaustion - GHSA-f886-m6hf-6m8v
@sdk-maintainer-bot sdk-maintainer-bot bot added missing-maintainer-discussion Used for automated community contribution checks. violating-contribution-guidelines Used for automated community contribution checks. labels Apr 10, 2026
@sdk-maintainer-bot
Copy link
Copy Markdown

This PR has been automatically closed. The referenced issue does not show a discussion between you and a maintainer.

To avoid wasted effort on both sides, please discuss your proposed approach in the issue first and wait for a maintainer to respond before opening a PR.

Please review our contributing guidelines for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Update to avoid brace-extansion issue by MaitreGEEK in #20205

🤖 This preview updates automatically when you update the PR.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 53ef9de. Configure here.

"@opentelemetry/semantic-conventions": "^1.40.0",
"@prisma/instrumentation": "7.6.0",
"@fastify/otel": "0.18.0",
"@fastify/otel": "0.18.1",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

yarn.lock not updated for dependency version bump

High Severity

The package.json bumps @fastify/otel to 0.18.1 (exact pin, no ^/~), but the yarn.lock still resolves @fastify/otel@0.18.0. This inconsistency means yarn install --frozen-lockfile (common in CI) will fail, and the intended security fix for the brace-expansion vulnerability won't actually take effect since the lockfile still pins the old version with the vulnerable transitive dependency. The yarn.lock needs to be regenerated to include the new version.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 53ef9de. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing-maintainer-discussion Used for automated community contribution checks. violating-contribution-guidelines Used for automated community contribution checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant