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.

Sign up for GitHub

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

Jump to bottom

Improve guac query vuln error message #1695

Merged
merged 1 commit into from Feb 12, 2024
Merged

Improve guac query vuln error message #1695

merged 1 commit into from Feb 12, 2024

Conversation

ridhoq
Copy link
Contributor

@ridhoq ridhoq commented Feb 9, 2024

Description of the PR

If guac query vuln fails, tell the user to run guacone certifier osv

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

@ridhoq
Improve guac query vuln error message
f89d3a8 
Signed-off-by: Ridwan Hoq <ridwanhoq@microsoft.com>
@kodiakhq kodiakhq bot merged commit e8ff763 into guacsec:main Feb 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lumjjb lumjjb lumjjb approved these changes

@pxp928 pxp928 pxp928 approved these changes

@jeffmendoza jeffmendoza Awaiting requested review from jeffmendoza jeffmendoza is a code owner

Assignees
No one assigned
Labels
size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ridhoq @lumjjb @pxp928