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.

Sign up for GitHub

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

Jump to bottom

Add audit alerts to exception list #1075

Merged
merged 2 commits into from
May 3, 2019
Merged

Add audit alerts to exception list #1075

merged 2 commits into from
May 3, 2019

Conversation

Gnito
Copy link
Contributor

@Gnito Gnito commented May 3, 2019 •
edited
Loading

yarn run audit script returned new vulnerability warnings.

Those vulnerable dependencies seemed to be used by libs in test and build phase.

Side note: we are in the process of updating sharetribe-scripts aka our Create React App (CRA) fork soonish. It should help with these warnings. (There are 2 updates coming, CRA v2.1.8 update and CRA v3.0.0 (hopefully). The latter was released a week ago.)

@Gnito Gnito force-pushed the add-audit-alerts-to-exception-list branch from 5626373 to 4c6b8c4 Compare May 3, 2019 18:57
@Gnito Gnito merged commit ec86dfa into master May 3, 2019
@Gnito Gnito deleted the add-audit-alerts-to-exception-list branch May 3, 2019 18:58
@Gnito Gnito mentioned this pull request May 8, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Gnito