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

Adding 'clair-scanner' cli tool as one of integration tools #442

Merged
merged 2 commits into from
Aug 7, 2017

Conversation

arminc
Copy link
Contributor

@arminc arminc commented Aug 6, 2017

I have created a spin-off from 'analyze-local-images' before it got deprecated, called clair-scanner. Clair-scanner uses Clair to check for vulnerabilities and blocks when there is one. But it also has an option to whitelist vulnerabilities which allows Clair to be used in a straight trough process as part of CI/CD pipeline.

Although clair-scanner is derived from 'analyze-local-images' the code is different and is maintained.

@arminc arminc changed the title Adding clair-scanner as an integration tool Adding 'clair-scanner' cli tool as one of integration tools Aug 6, 2017
@@ -22,4 +22,6 @@ This document tracks projects that integrate with Clair. [Join the community](ht

[Portus](http://port.us.org/features/6_security_scanning.html#coreos-clair): an authorization service and frontend for Docker registry (v2).

[clair-scanner](https://github.com/arminc/clair-scanner): a spin-off from 'analyze-local-images' who blocks on vulnerabilities with whitelist possibility
Copy link
Contributor

Choose a reason for hiding this comment

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

This sentence confused me for a bit. How about something along the lines of:

a project similar to 'analyze-local-images' with a whitelisting feature

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had that but thought it's not clear that it blocks... I agree this is much simpler

Copy link
Contributor

@jzelinskie jzelinskie left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for this project :D

@jzelinskie jzelinskie merged commit 2f08cf5 into quay:master Aug 7, 2017
KeyboardNerd pushed a commit to KeyboardNerd/clair that referenced this pull request Feb 2, 2018
Adding 'clair-scanner' cli tool as one of integration tools
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants