Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

Add option to ignore vulnerabilities by the id #4

Open
rtfpessoa opened this issue Oct 6, 2018 · 2 comments
Open

Add option to ignore vulnerabilities by the id #4

rtfpessoa opened this issue Oct 6, 2018 · 2 comments
Labels
Hacktoberfest Entry level issues for https://hacktoberfest.digitalocean.com/ help wanted

Comments

@rtfpessoa
Copy link
Owner

Add --ignore option to allow passing a comma separated list of ids to be ignored in the analysis.
@rtfpessoa rtfpessoa added help wanted Hacktoberfest Entry level issues for https://hacktoberfest.digitalocean.com/ labels Oct 6, 2018
@rtfpessoa rtfpessoa mentioned this issue Oct 6, 2018
Closed
@harman28
Copy link

harman28 commented Oct 6, 2018 •
edited
Loading

What kind of IDs would these be? Package names in some form?
Could you give an example of the desired usage?

@rtfpessoa
Copy link
Owner Author

rtfpessoa commented Oct 6, 2018 •
edited
Loading

In the output if you request -f json you should receive an id field for each vulnerability. e.g.: nodesecurity:npm:brace-expansion:2017-04-25

sundus-y added a commit to sundus-y/dependency_spy that referenced this issue Oct 29, 2018
@sundus-y
Add option to ignore vulnerabilities by the id. (rtfpessoa#4)
0b3ccd7 
* Add --ignore option to pass in list of vulnerability ids to ignore.
* --ignore is a comma separated list.
* Refactored the check method argumment to options hash. This was done
  to avoid listing all options one by one in the method argument and
  also RuboCop was failing for 'Avoid parameter lists longer than 5
  parameters.'
* Added rspec test around the --ignore option.
@sundus-y sundus-y mentioned this issue Oct 29, 2018
Closed
sundus-y added a commit to sundus-y/dependency_spy that referenced this issue Oct 29, 2018
@sundus-y
Add option to ignore vulnerabilities by the id. (rtfpessoa#4)
ac863ae 
* Fix typo.
sundus-y added a commit to sundus-y/dependency_spy that referenced this issue Oct 29, 2018
@sundus-y
Add option to ignore vulnerabilities by the id. (rtfpessoa#4)
2e0685a 
* Fix Rubocop
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Hacktoberfest Entry level issues for https://hacktoberfest.digitalocean.com/ help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rtfpessoa @harman28