chore: resolve open dependabot security alerts#40
Conversation
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
There was a problem hiding this comment.
Code Review
This pull request updates the uuid dependency to version 11.1.1 and postcss to 8.5.14, and adds a postcss override in package.json. The reviewer recommends moving postcss from devDependencies to dependencies to ensure it is available during production-only installs, as it is now a production-critical dependency due to the override and its use by production dependencies like next.
- fast-uri <=3.1.1 -> >=3.1.2 (high, alert #77, #78) Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
There was a problem hiding this comment.
Pull request overview
This PR addresses Dependabot-reported security vulnerabilities by updating dependency versions and enforcing safer transitive resolution via npm overrides.
Changes:
- Bumped
uuidfrom^11.1.0to^11.1.1. - Promoted
postcssto a direct dependency and added an override to deduplicate to the rootpostcssversion. - Added an override to force
fast-urito resolve to>=3.1.2.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Updates dependency versions and adds npm overrides for vulnerable transitive packages. |
| package-lock.json | Locks updated resolved versions for uuid, postcss, and fast-uri, reflecting the new override behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
…alerts Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUpgrades OpenTelemetry packages from ChangesOpenTelemetry upgrade and API adaptation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Around line 21-28: Update two OpenTelemetry experimental package versions in
package.json to match `@vercel/otel` 2.1.3 compatibility requirements: change
`@opentelemetry/api-logs` from ^0.202.0 to ^0.205.0 and change
`@opentelemetry/sdk-logs` from ^0.202.0 to ^0.205.0. Additionally, add
`@opentelemetry/sdk-trace-base` as a new direct dependency with version ^2.1.0 in
the dependencies section, as it is currently only installed transitively through
other packages.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 91d75b4e-4474-4af8-b271-7b746b9c145f
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (3)
package.jsonsrc/instrumentation.tssrc/libs/open-feature/telemetry-hook.ts
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Summary
Dependabot Alerts Resolved
fast-uri>=3.1.2(host confusion via percent-encoded authority delimiters)fast-uri>=3.1.2(path traversal via percent-encoded dot segments)uuid>=11.1.1(missing buffer bounds check)postcss>=8.5.10(XSS); moved from devDependencies to dependencies for correct override resolution