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

Keep Python and JS requirements up to date / off insecure versions #500

Closed
Osmose opened this issue Feb 9, 2017 · 3 comments
Closed

Keep Python and JS requirements up to date / off insecure versions #500

Osmose opened this issue Feb 9, 2017 · 3 comments
Assignees

Comments

@Osmose
Copy link
Contributor

Osmose commented Feb 9, 2017

#286 recommends we use an automated tool of some kind to monitor our dependencies and either keep them up to date, or at least alert us when they're out of date or have a security vulnerability.

After looking at their suggested tools, I think our best bet is:

Beyond alternatives to the above plan, the main open question is how to run nsp. Running it in CI won't ensure we get notified when a vulnerability is found until we make a PR or commit, but we don't have any regular jobs that just run and monitor the code.

@mozilla/normandy Thoughts?

@mythmon
Copy link
Contributor

mythmon commented Feb 10, 2017

I'm ok with both of these tools.

In the absence of an automated, regular way to run nsp, I think running in CI is a good first step while we explore other options. We make commits to this repo pretty often: in the last year our longest gap was 6 days (around Christmas). In the past month out longest gap has been 2 days.

@chartjes
Copy link
Contributor

chartjes commented Feb 10, 2017 via email

Osmose pushed a commit to Osmose/normandy that referenced this issue Feb 15, 2017
@Osmose Osmose self-assigned this Feb 15, 2017
@Osmose Osmose added this to the Sprint 5 - February 6th - February 18th, 2017 milestone Feb 15, 2017
Osmose added a commit that referenced this issue Feb 16, 2017
#500: Add NSP to lint command in CI.
@Osmose
Copy link
Contributor Author

Osmose commented Feb 16, 2017

Requires.io is configured for the team/project, and NSP landed in prod, although it's having some issues: #520

I think we can call this closed.

@Osmose Osmose closed this as completed Feb 16, 2017
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

No branches or pull requests

3 participants