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

Bump dependencies to resolve vulnerabilities #1164

Merged
merged 5 commits into from
Jun 14, 2024
Merged

Conversation

rhystmills
Copy link
Contributor

@rhystmills rhystmills commented Jun 11, 2024

What does this change?

This bumps some dependencies in order to reduce the number of high priority vulnerabilities reported by Snyk.

It reduces the number of Scala vulnerabilities from 6 to 2. Of the remaining two:

  1. One is present in the latest release of Play, so we can't solve it for now.
  2. One is present in the latest version of panda so we'll need to sort it there first. More details are available in Snyk.

The JS changes don't solve the vulnerabilities, but they do bring ts-loader to its latest version (where the vuln is still present).

How to test

Deploy to CODE and see if everything appears to be working normally.

@rhystmills rhystmills marked this pull request as ready for review June 12, 2024 15:43
@rhystmills rhystmills requested a review from a team as a code owner June 12, 2024 15:43
Copy link
Contributor

@Georges-GNM Georges-GNM left a comment

Choose a reason for hiding this comment

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

Checked this on code, had a bit of a look/click around, looked ok to me 👍

@rhystmills rhystmills merged commit 71f7e7c into main Jun 14, 2024
1 of 2 checks passed
@rhystmills rhystmills deleted the rm/bump-dependencies branch June 14, 2024 10:04
@prout-bot
Copy link

Seen on PROD (merged by @rhystmills 6 minutes and 25 seconds ago) Please check your changes!

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

Successfully merging this pull request may close these issues.

3 participants