Skip to content

[Snyk] Security upgrade vscode-extension-telemetry from 0.1.6 to 0.4.5#23

Open
smilers wants to merge 1 commit intomainfrom
snyk-fix-99cc2d136b60aec3f69df926d0dd49f3
Open

[Snyk] Security upgrade vscode-extension-telemetry from 0.1.6 to 0.4.5#23
smilers wants to merge 1 commit intomainfrom
snyk-fix-99cc2d136b60aec3f69df926d0dd49f3

Conversation

@smilers
Copy link

@smilers smilers commented Feb 2, 2024

This PR was automatically created by Snyk using the credentials of a real user.


Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • extensions/ql-vscode/package.json
    • extensions/ql-vscode/package-lock.json

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Breaking Change Exploit Maturity
high severity 696/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
Regular Expression Denial of Service (ReDoS)
SNYK-JS-SEMVER-3247795
No Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages
Package name: vscode-extension-telemetry The new version differs by 81 commits.
  • 21d7c13 Missed a place bumping the version
  • 41bc647 Update version for release
  • c561107 Lower target to support more legacy codebases
  • 4911887 Fix #88
  • 1551186 Update build to node LTS
  • 081c624 Remove whitespace expansion due to perf reasons
  • 188ee72 Merge pull request #73 from radeksimko/f-collect-arch
  • ddeafdb common.arch -> common.nodeArch
  • 4d7a45b common: Collect architecture as a common property
  • bdbab89 Remove first party explicitness from readme
  • 068ddd9 Fix compilation
  • 1ca205c Update level enum
  • e0f1cca Bump version to prepare for a release
  • 389b8b2 Fix #76
  • 0e1a889 Switch to npm 6
  • 1099714 Update package.json with new esbuild
  • 7174c44 Merge pull request #75 from radeksimko/f-raw-telemetry-event
  • 92d1291 rename: TelemetryRawEventProperties -> RawTelemetryEventProperties
  • c3ea7fc simplify object notation
  • c4d17f1 Add codespaces as a remote authority
  • 91e1e18 fix typo Telemtry -> Telemetry
  • 7d2d3e4 Introduce 'sendRawTelemetryEvent'
  • bb8286d Run on macos latest
  • 7bf72ee Update ansi regex

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report

🛠 Adjust project settings

📚 Read more about Snyk's upgrade and patch logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Regular Expression Denial of Service (ReDoS)

…-lock.json to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-SEMVER-3247795
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

PR Type: Enhancement

PR Summary: This PR addresses a security vulnerability by upgrading the 'vscode-extension-telemetry' package from version 0.1.6 to 0.4.5. The upgrade mitigates the risk associated with a Regular Expression Denial of Service (ReDoS) vulnerability.

Decision: Comment

📝 Type: 'Enhancement' - not supported yet.
  • Sourcery currently only approves 'Typo fix' PRs.
✅ Issue addressed: this change correctly addresses the issue or implements the desired feature.
No details provided.
📝 Complexity: the changes are too large or complex for Sourcery to approve.
  • Unsupported files: the diff contains files that Sourcery does not currently support during reviews.

General suggestions:

  • Ensure that the upgrade does not introduce any breaking changes that could affect the telemetry functionality within the extension.
  • Verify that all new features or deprecations introduced in the upgraded package are compatible with the current implementation of the extension.
  • Consider reviewing the commit history of the upgraded package to understand the scope of changes and their potential impact on the extension.

Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨

Share Sourcery

Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants