-
Notifications
You must be signed in to change notification settings - Fork 3.3k
chore(deps): remove sharp dependency resolution #32378
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
Conversation
See the guidelines for reviewing dependency updates for info on how to review dependency update PRs. |
afdac5f
to
e9c4c52
Compare
Tracked down where this was introduced: edccfc9 sharp was a sub dependency of favicons, but I don't see that used anymore. I checked that the favicon is working after this change still. I don't think we need this resolution anymore. |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
cypress
|
Project |
cypress
|
Branch Review |
renovate/npm-sharp-vulnerability
|
Run status |
|
Run duration | 19m 26s |
Commit |
|
Committer | Jennifer Shehane |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
10
|
|
1102
|
|
0
|
|
26661
|
View all changes introduced in this branch ↗︎ |
UI Coverage
45.11%
|
|
---|---|
|
186
|
|
157
|
Accessibility
97.71%
|
|
---|---|
|
4 critical
8 serious
2 moderate
2 minor
|
|
110
|
This PR contains the following updates:
0.29.3
->0.32.6
GitHub Vulnerability Alerts
CVE-2022-29256
There's a possible vulnerability in logic that is run only at
npm install
time when installing versions ofsharp
prior to the latest v0.30.5.This is not part of any runtime code, does not affect Windows users at all, and is unlikely to affect anyone that already cares about the security of their build environment. However, out of an abundance of caution, I've created this advisory.
If an attacker has the ability to set the value of the
PKG_CONFIG_PATH
environment variable in a build environment then they might be able to use this to inject an arbitrary command atnpm install
time.I've used the Common Vulnerability Scoring System (CVSS) calculator to determine the maximum possible impact, which suggests a "medium" score of 5.9, but for most people the real impact will be dealing with the noise from automated security tooling that this advisory will bring.
AV:L/AC:L/PR:H/UI:R/S:U/C:H/I:H/A:H/E:P/RL:O/RC:C/CR:X/IR:X/AR:X/MAV:X/MAC:X/MPR:X/MUI:R/MS:X/MC:X/MI:X/MA:X
This problem was fixed in commit a6aeef6 and published as part of
sharp
v0.30.5.Thank you very much to @dwisiswant0 for the responsible disclosure.
Remember: if an attacker has control over environment variables in your build environment then you have a bigger problem to deal with than this issue.
GHSA-54xq-cgqr-rpm3
Overview
sharp uses libwebp to decode WebP images and versions prior to the latest 0.32.6 are vulnerable to the high severity GHSA-j7hp-h8jx-5ppr.
Who does this affect?
Almost anyone processing untrusted input with versions of sharp prior to 0.32.6.
How to resolve this?
Using prebuilt binaries provided by sharp?
Most people rely on the prebuilt binaries provided by sharp.
Please upgrade sharp to the latest 0.32.6, which provides libwebp 1.3.2.
Using a globally-installed libvips?
Please ensure you are using the latest libwebp 1.3.2.
Possible workaround
Add the following to your code to prevent sharp from decoding WebP images.
Release Notes
lovell/sharp (sharp)
v0.32.6
Compare Source
v0.32.5
Compare Source
v0.32.4
Compare Source
v0.32.3
Compare Source
v0.32.2
Compare Source
v0.32.1
Compare Source
v0.32.0
Compare Source
v0.31.3
Compare Source
v0.31.2
Compare Source
v0.31.1
Compare Source
v0.31.0
Compare Source
v0.30.7
Compare Source
v0.30.6
Compare Source
v0.30.5
Compare Source
v0.30.4
Compare Source
v0.30.3
Compare Source
v0.30.2
Compare Source
v0.30.1
Compare Source
v0.30.0
Compare Source
Configuration
📅 Schedule: Branch creation - "" in timezone America/New_York, Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.