-
Notifications
You must be signed in to change notification settings - Fork 156
fix: address devDependencies audit vulnerabilities (sassdoc, gulp-shell) #16534
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
base: master
Are you sure you want to change the base?
Conversation
…vulnerability Co-authored-by: Lipata <2621802+Lipata@users.noreply.github.com>
|
@copilot I've updated all the Angular packages, using the |
Co-authored-by: Lipata <2621802+Lipata@users.noreply.github.com>
Reverted the Angular package updates and fixed the devDependencies vulnerabilities in commit c38e08b:
Audit results: 19 vulnerabilities (4 moderate, 15 high) → 3 low severity vulnerabilities (sassdoc-extras prototype pollution with no patched version available) Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot, please fix the misalignment between pacakge.json and package-lock.json. |
…es format Co-authored-by: Lipata <2621802+Lipata@users.noreply.github.com>
Fixed in commit 033a429. The nested overrides format wasn't working correctly, so I simplified it to direct package overrides which properly align package.json and package-lock.json. npm audit now reports only 3 low severity vulnerabilities. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Closes #16533
Fixes npm audit vulnerabilities in devDependencies (sassdoc, gulp-shell) by removing unused packages and adding npm overrides for transitive dependencies.
Changes:
Removed gulp-shell: The package was not used anywhere in the codebase and had a high severity vulnerability (lodash.template command injection - GHSA-35jh-r3h4-6jhm)
Added npm overrides for sassdoc vulnerabilities: Fixed transitive dependency vulnerabilities in sassdoc using simplified override format:
Fixed package.json and package-lock.json alignment: Simplified the overrides format from nested objects to direct package overrides to ensure proper alignment between package.json and package-lock.json.
Audit Results:
The remaining 3 low severity vulnerabilities are in sassdoc-extras itself (prototype pollution), for which there is no patched version available. These are in devDependencies only and do not affect the production library.
Note: Angular package updates are handled separately in PR #16535 via
ng update.Additional information (check all that apply):
Checklist:
feature/README.MDupdates for the feature docsREADME.MDCHANGELOG.MDupdates for newly added functionalityng updatemigrations for the breaking changes (migrations guidelines)Original prompt
This section details on the original issue you should resolve
<issue_title>Npm audit failure in the 20.1.x branch</issue_title>
<issue_description>## Description
There are some severe audit issues, which gives us problems in our pipeline.
Steps to reproduce
Result