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

ui: JavaScript Security Vulnerabilities #41538

Merged
merged 7 commits into from
Oct 14, 2019
Merged

ui: JavaScript Security Vulnerabilities #41538

merged 7 commits into from
Oct 14, 2019

Conversation

nathanstilwell
Copy link
Contributor

@nathanstilwell nathanstilwell commented Oct 11, 2019

Resolving vulnerabilities alerted by Github,

  • handlebars
  • mem
  • lodash-es
  • randomatic

Unable to resolve a vulnerability reported by synk as no remediation exists currently. Also unable to resolve the github alert for webpack-dev-server without upgrading to Weback version 4.

Nathan Stilwell added 7 commits October 11, 2019 12:13
Upgrading webpack-dashboard to resolve a high severity vulnerability in
handlbars

Release note (admin ui change): webpack-dashboard (used in developing admin ui)
upgraded
I am removing a resolution I created for axios. I don't recall if this
was done in error or if new vulnerability reports have replaced old
ones, but as it currently stands
(https://app.snyk.io/vuln/SNYK-JS-AXIOS-472067) there is no remediation
path available for this vulnerability.

For the moment I am correcting our dependencies since the previous
resolution left `analytics-node` without axios as a dependency
potentially breaking segment analytics. By removing the resolution and
upgrading to the latest stable version of `analytics-node`, I believe
this is the best we can do for the moment.

Release note: None
Resolving a github security alert for transient dependency on a version
of lodash-es with a critial vulnerability

references:
- lodash/lodash#4336
- https://snyk.io/vuln/SNYK-JS-LODASH-73638

Release note: None
Bumping version of mocha to remove a dependency on `mem` which has a
moderate vulnerability alerted by github

Release note: None
this removes randomatic as a dependency, which had a security
vulnerabilty alerted by github

Release note: None
Upgrading karma to remediate a security vulnerability reported by npm
audit (https://www.npmjs.com/advisories/786)

Release note: None
Adding dependency upgrades to yarn-vendor

Release note: None
@nathanstilwell nathanstilwell requested a review from a team October 11, 2019 19:22
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@vilterp vilterp left a comment

Choose a reason for hiding this comment

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

As long as you have built the UI and clicked around locally, this should be good to go.

Planning on backporting into the release?

@nathanstilwell
Copy link
Contributor Author

nathanstilwell commented Oct 11, 2019

@vilterp The app, tests, and webpack dev all seem to behave correctly. How do I backport into the release? I have heard this mentioned, but haven't seen the docs yet.
Backported - #41539

@nathanstilwell
Copy link
Contributor Author

bors r+

craig bot pushed a commit that referenced this pull request Oct 14, 2019
41538: ui: JavaScript Security Vulnerabilities r=nathanstilwell a=nathanstilwell

Resolving vulnerabilities alerted by Github,
+ handlebars
+ mem
+ lodash-es
+ randomatic

Unable to resolve a vulnerability [reported by synk](https://app.snyk.io/vuln/SNYK-JS-AXIOS-472067) as no remediation exists currently. Also unable to resolve the github alert for `webpack-dev-server` without upgrading to Weback version 4. 

Co-authored-by: Nathan Stilwell <[email protected]>
@craig
Copy link
Contributor

craig bot commented Oct 14, 2019

Build succeeded

@craig craig bot merged commit 816c326 into cockroachdb:master Oct 14, 2019
@nathanstilwell nathanstilwell deleted the javascript-security-vulnerabilities branch October 14, 2019 16:22
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.

3 participants