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

fix(queries): fixed Docker queries related to issues 5115, 5116, and 5118 #5295

Merged
merged 8 commits into from
Sep 28, 2022
Merged

fix(queries): fixed Docker queries related to issues 5115, 5116, and 5118 #5295

merged 8 commits into from
Sep 28, 2022

Conversation

cxAndreFelicidade
Copy link
Contributor

@cxAndreFelicidade cxAndreFelicidade commented May 2, 2022 •
edited
Loading

Closes #5115, closes #5116, closes #5118

Proposed Changes

  • On issue 5115: Query was changed and files refactored.
  • On issue 5116: Query description was changed.
  • On issue 5118: Query was changed and files refactored.

I submit this contribution under the Apache-2.0 license.

@cxAndreFelicidade cxAndreFelicidade added docker Docker query community Community contribution accuracy Query accuracy (FN and FP) labels May 2, 2022
@cxAndreFelicidade cxAndreFelicidade added this to the Query Accuracy milestone May 2, 2022
@cxAndreFelicidade cxAndreFelicidade self-assigned this May 2, 2022
@kicsbot
Copy link
Contributor

kicsbot commented May 2, 2022

Scan submitted to Checkmarx

This was referenced May 2, 2022
Closed
Closed
Closed
@kicsbot
Copy link
Contributor

kicsbot commented May 2, 2022

Logo
Checkmarx SAST - Scan Summary & Details

Cx-SAST Summary

Total of 5 vulnerabilities
High 0 High
Medium 0 Medium
Low 5 Low
Info 0 Info

Violation Summary

No policy violation found

@nunoocx nunoocx requested a review from rjegoncalves May 2, 2022 15:47
@rafaela-soares rafaela-soares changed the title Fix(queries): fixed Docker queries related to issues 5115, 5116, and 5118 fix(queries): fixed Docker queries related to issues 5115, 5116, and 5118 May 5, 2022
@kaplanlior kaplanlior removed the community Community contribution label May 8, 2022
joaorufi
joaorufi previously approved these changes May 20, 2022
View reviewed changes
@cxMiguelSilva cxMiguelSilva self-assigned this Sep 26, 2022
rafaela-soares
rafaela-soares approved these changes Sep 28, 2022
View reviewed changes
Copy link
Contributor

@rafaela-soares rafaela-soares left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@rafaela-soares rafaela-soares merged commit b8403d6 into master Sep 28, 2022
@rafaela-soares rafaela-soares deleted the query/issues5115_5116_5118 branch September 28, 2022 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rafaela-soares rafaela-soares rafaela-soares approved these changes

@joaorufi joaorufi joaorufi left review comments

@joaoReigota1 joaoReigota1 Awaiting requested review from joaoReigota1

@rjegoncalves rjegoncalves Awaiting requested review from rjegoncalves

Assignees

@cxMiguelSilva cxMiguelSilva

@cxAndreFelicidade cxAndreFelicidade

Labels
accuracy Query accuracy (FN and FP) docker Docker query
Projects
None yet
Milestone
Query Accuracy
6 participants
@cxAndreFelicidade @kicsbot @rafaela-soares @joaorufi @kaplanlior @cxMiguelSilva