Skip to content

Add Certifier cmd to guacone#246

Merged
kodiakhq[bot] merged 5 commits intoguacsec:mainfrom
pxp928:certifierCmd
Dec 7, 2022
Merged

Add Certifier cmd to guacone#246
kodiakhq[bot] merged 5 commits intoguacsec:mainfrom
pxp928:certifierCmd

Conversation

@pxp928
Copy link
Copy Markdown
Collaborator

@pxp928 pxp928 commented Nov 22, 2022

Add Certifier command to guacone to manually call the certifier and kick off the vulnerability certification process for testing.

This will run through the current graphDB and look for all packages to run the OSV query against. If a vulnerability is found, an attestation along with a vulnerability node is generated.

Signed-off-by: pxp928 <parth.psu@gmail.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>
Signed-off-by: pxp928 <parth.psu@gmail.com>
@pxp928 pxp928 marked this pull request as ready for review December 5, 2022 03:06
Copy link
Copy Markdown
Contributor

@lumjjb lumjjb left a comment

Choose a reason for hiding this comment

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

minor nits, else LGTM

Comment thread cmd/guacone/cmd/certifier.go Outdated
Comment thread cmd/guacone/cmd/certifier.go Outdated
Signed-off-by: pxp928 <parth.psu@gmail.com>
@pxp928 pxp928 requested a review from lumjjb December 5, 2022 18:23
Copy link
Copy Markdown
Contributor

@lumjjb lumjjb left a comment

Choose a reason for hiding this comment

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

lgtm

@kodiakhq kodiakhq bot merged commit 13b9a49 into guacsec:main Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants