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

Upgrade jasmine-browser-runner dependencies (particularly "send") #3162

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

Gweaton
Copy link
Contributor

@Gweaton Gweaton commented Sep 12, 2024

to fix a security vulnerability.

We had a medium security vulnerability because jasmine-browser-runner depends on express 4.20.0 which depends on serve-static 0.16.0 which depends on send 0.18.0, which is currently vulnerable due to GHSA-m6fv-jmcg-4jfg.

The changes to yarn.lock were generated by running yarn upgrade jasmine-browser-runner to ensure it's using the patched version (0.19.0) of the send dependency.

This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

to fix a security vulnerability.

We had a medium security vulnerability because jasmine-browser-runner
depends on express 4.20.0 which depends on serve-static 0.16.0 which
depends on send 0.18.0, which is currently vulnerable due to
GHSA-m6fv-jmcg-4jfg. We pinned the express
dependency to resolve this but now that a new version of serve-static
has been released, we can unpin this.

The changes to `yarn.lock` were generated by running `yarn upgrade
jasmine-browser-runner` to ensure it's using the patched version
(0.19.0) of the `send` dependency.
@Gweaton Gweaton force-pushed the fix-send-vulnerability branch from dbf7b52 to 99c656b Compare September 12, 2024 10:39
@Gweaton Gweaton merged commit 04baec6 into main Sep 12, 2024
15 checks passed
@Gweaton Gweaton deleted the fix-send-vulnerability branch September 12, 2024 10:59
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