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 @slack/client from 4.12.0 to 5.0.0 #241

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

Conversation

snyk-bot
Copy link

@snyk-bot snyk-bot commented Sep 5, 2021

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:
    • package.json

Vulnerabilities that will be fixed

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

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

Commit messages
Package name: @slack/client The new version differs by 250 commits.
  • 891307e Publish
  • 55624fd lerna generated SHAs
  • a76b3ee Merge pull request #756 from slackapi/dev-v5
  • 09d6d6f fix a typescript issue from changing from type aliases to interfaces
  • 4d3133c remove package locks from examples
  • c3e5c8b update examples for testing
  • d40c883 remove file accidentally added in merge
  • 0d1cd79 Merge branch 'master' into dev-v5
  • b6dc555 Merge pull request #755 from aoberoi/find-and-replace
  • 65788d2 Merge pull request #754 from shanedewael/local-development-tutorial
  • 81be95b its amazing what you can find with a little find and replace
  • 8c98232 Consolidate cta
  • 53861c9 Update docs/_tutorials/local_development.md
  • 74b8e77 Update docs/_tutorials/local_development.md
  • 200a430 Update docs/_tutorials/local_development.md
  • 95cd749 Changes
  • 2ee0224 Merge pull request #752 from aoberoi/scheduled-messages-v5
  • 695984a Merge pull request #753 from aoberoi/issue-templates-v5
  • aa126f0 Merge pull request #751 from aoberoi/reference-examples
  • 5f7a654 Add table of contents
  • e73ad7d Fix links on READMEs
  • d1eb873 Local development tutorial
  • 17318b2 add warning to maintainers guide
  • dd84a72 update issue template to ask for relevant packages

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

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-AXIOS-1579269
@optionsome optionsome changed the base branch from master to v2 January 21, 2022 09:06
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