Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Snyk] Security upgrade keytar from 4.2.1 to 4.3.2 #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snyk-bot
Copy link
Contributor

@snyk-bot snyk-bot commented Jun 7, 2022

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

merge advice

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • app/package.json
    • app/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-ANSIREGEX-1583908
No Proof of Concept

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

Commit messages
Package name: keytar The new version differs by 39 commits.
  • 22c0b7b 4.3.2
  • 3dd3e05 Merge pull request #154 from atom/workaround-preinstall-issue
  • 27f6b87 switch to custom prebuild-install package which has an explicit electron v4 check
  • c938d5a 4.3.1
  • a5c7fe4 Merge pull request #152 from nornagon/node-12
  • 8841615 fix: Handle => Local for node 12 support
  • 5a5854b Update README.md
  • 6672ed4 Merge pull request #150 from atom/docs-tidy-up
  • 338d1e1 reorder this bit to emphasize findCredentials
  • 6cb9ad5 drop mention of account
  • c10e150 Merge pull request #145 from atom/testing-for-electron-4
  • 3f6dd78 upgrade node-abi to support newer versions of Node and Electron
  • 4e08707 prebuild with Electron v4
  • 2ccebc0 Merge pull request #136 from atom/address-cryptiles-vulnerability
  • 0c18375 update node-gyp to address upstream CVE
  • 581c974 4.3.0
  • dcb7416 Merge pull request #126 from atom/add-newer-nodes-for-prebuild
  • 86731a9 upgrade prebuild to latest version
  • f460390 add specific node-abi dependency so we can prebuild against Electron 3
  • abab53b add newer node tags to prebuild scripts
  • e48b67d Merge pull request #119 from atom/bump-to-use-node-10
  • da73b19 use Node 10 on CI for testing and packaging
  • f2d481f Merge pull request #118 from atom/audit-cleanup
  • ea93b40 automated npm audit fix

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:

🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

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.

1 participant