-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Chore/dep patch #968
Chore/dep patch #968
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, just need to update gen3-ui-component and guppy
@@ -27,8 +27,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \ | |||
&& rm -rf /var/lib/apt/lists/* \ | |||
&& ln -sf /dev/stdout /var/log/nginx/access.log \ | |||
&& ln -sf /dev/stderr /var/log/nginx/error.log \ | |||
&& npm install -g npm@7 \ | |||
&& npm config set maxsockets 5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curious why we don't need this anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this issue has been addressed in a newer version of npm npm/cli#2977 (comment)
Jira Ticket: FRC-35
deployed in https://mingfei.planx-pla.net/
Update dependencies to patch FedRAMP vulnerabilities
Dependency updates