-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Github security warning for hoek
#11699
Comments
To fully rectify the issue for all users, this PR (nodejs/node-gyp#1471) is required for node-gyp followed by a release and version bump in node-sass and finally a version bump on the CLIs end. |
@clydin how about we fix this by moving to the new canonical sass compiler which is npmjs.com/sass |
Sounds good. Would we need to target 7.0 due to the potential for behavior differences? This PR for the webpack loader would also be blocking (webpack-contrib/sass-loader#573) unless we wanted to make our own. |
This seems to fix it: |
Should address some See sass/node-sass#2355 for context. Fix angular#11699 Related to angular#10963
See sass/node-sass#2355 for context. Fix angular#11699 Related to angular#10963
See sass/node-sass#2355 for context. Fix angular#11699 Related to angular#10963
See sass/node-sass#2355 for context. Fix #11699 Related to #10963
See sass/node-sass#2355 for context. Fix angular#11699 Related to angular#10963
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug Report or Feature Request (mark with an
x
)Versions
Angular CLI: 6.1.1
Node: 10.7.0
OS: linux x64
Angular: 6.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package Version
@angular-devkit/architect 0.6.8
@angular-devkit/build-angular 0.6.8
@angular-devkit/build-optimizer 0.6.8
@angular-devkit/core 0.6.8
@angular-devkit/schematics 0.7.1
@angular/cdk 6.4.1
@angular/cli 6.1.1
@angular/material 6.4.1
@ngtools/webpack 6.0.8
@schematics/angular 0.7.1
@schematics/update 0.7.1
rxjs 6.2.2
typescript 2.7.2
webpack 4.8.3
Repro steps
You can use
yarn why
to see why we have hoekThe log given by the failure
This links to: https://nvd.nist.gov/vuln/detail/CVE-2018-3728
Desired functionality
Remove or bump dependency on node-sass to remove dependency on reported vulnerability in
hoek
.The text was updated successfully, but these errors were encountered: