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

Run audit as shell script instead of as single line command #610

Merged
merged 5 commits into from Jun 22, 2020

Conversation

mozillazg
Copy link
Collaborator

Fixes #604

@codecov
Copy link

codecov bot commented Apr 19, 2020

Codecov Report

Merging #610 into master will decrease coverage by 0.17%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #610      +/-   ##
==========================================
- Coverage   65.29%   65.12%   -0.18%     
==========================================
  Files          12       12              
  Lines        1144     1078      -66     
==========================================
- Hits          747      702      -45     
+ Misses        357      342      -15     
+ Partials       40       34       -6     
Impacted Files Coverage Δ
check/controls.go 72.04% <ø> (-1.96%) ⬇️
check/check.go 80.00% <100.00%> (+6.61%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35cf28c...680bc64. Read the comment docs.

@lizrice
Copy link
Contributor

lizrice commented May 14, 2020

Sorry for being slow on this. From a very quick glance it looks similar to what @yoavrotems did with aquasecurity/bench-common#87

@mozillazg
Copy link
Collaborator Author

@lizrice Should i close this PR or use similar codes from aquasecurity/bench-common#87 ?

@mozillazg
Copy link
Collaborator Author

@yoavrotems ping

Copy link
Contributor

@yoavrotems yoavrotems left a comment

Choose a reason for hiding this comment

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

Sorry for the long time to response, I had to check some things about changes with the struct.
It seems like all is good ! :)
A very good PR and thank for your contribution

@yoavrotems yoavrotems merged commit c7b518e into aquasecurity:master Jun 22, 2020
caruccio pushed a commit to getupcloud/kube-bench that referenced this pull request Feb 19, 2021
…rity#610)

* Run audit as shell script instead of as single line command

* Rename runExecCommands to runAudit

* Fix tests

Co-authored-by: Liz Rice <liz@lizrice.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support run tests which have complex shell commands
3 participants