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

Adds additional dockerfile policies #984

Merged
merged 10 commits into from
Sep 11, 2021
Merged

Adds additional dockerfile policies #984

merged 10 commits into from
Sep 11, 2021

Conversation

rishidhamija
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Aug 12, 2021 •
edited
Loading

Codecov Report

Merging #984 (b72c340) into master (1269301) will decrease coverage by 0.35%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #984      +/-   ##
==========================================
- Coverage   78.58%   78.22%   -0.36%     
==========================================
  Files         224      237      +13     
  Lines        5650     6292     +642     
==========================================
+ Hits         4440     4922     +482     
- Misses        930     1067     +137     
- Partials      280      303      +23
Impacted Files Coverage Δ
pkg/iac-providers/kustomize/v3/load-file.go 0.00% <0.00%> (-100.00%) ⬇️
pkg/iac-providers/kustomize/v3/load-dir.go 0.00% <0.00%> (-71.70%) ⬇️
pkg/cli/run.go 87.65% <0.00%> (-1.24%) ⬇️
pkg/policy/types.go 100.00% <0.00%> (ø)
pkg/results/types.go 100.00% <0.00%> (ø)
pkg/http-server/remote-repo.go 71.01% <0.00%> (ø)
pkg/iac-providers/kustomize.go 100.00% <0.00%> (ø)
pkg/iac-providers/output/types.go 100.00% <0.00%> (ø)
pkg/iac-providers/kubernetes/v1/normalize.go 89.74% <0.00%> (ø)
pkg/iac-providers/terraform/commons/load-dir.go 78.50% <0.00%> (ø)
... and 22 more

@cesar-rodriguez cesar-rodriguez changed the title new docker policies Adds additional dockerfile policies Aug 14, 2021
@cesar-rodriguez cesar-rodriguez added the policy Issue concerning policy maintainers. label Aug 20, 2021
harkirat22
harkirat22 reviewed Aug 25, 2021
View reviewed changes
pkg/policies/opa/rego/docker/docker_run/multipleRunAddCopyInstructions.rego Outdated
@@ -0,0 +1,23 @@
package accurics
Copy link
Contributor

Choose a reason for hiding this comment

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

could you please verify if this policy is complete or you missed something

gaurav-gogia
gaurav-gogia previously approved these changes Aug 30, 2021
View reviewed changes
@sonarcloud
Copy link

sonarcloud bot commented Sep 1, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cesar-rodriguez cesar-rodriguez merged commit eaf5927 into tenable:master Sep 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rchanger Rchanger Rchanger left review comments

@harkirat22 harkirat22 harkirat22 approved these changes

@gaurav-gogia gaurav-gogia gaurav-gogia left review comments

Assignees

@rishidhamija rishidhamija

Labels
policy Issue concerning policy maintainers.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rishidhamija @gaurav-gogia @harkirat22 @Rchanger @cesar-rodriguez