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

updating s3 bucket policies for v4 aws provider #1175

Merged

Conversation

harkirat22
Copy link
Contributor

@harkirat22 harkirat22 commented Feb 28, 2022

This PR addresses the changes that happened for S3 bucket with new version of AWS provider.
This PR also resolves #1162

@sonarcloud
Copy link

sonarcloud bot commented Feb 28, 2022

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

@codecov
Copy link

codecov bot commented Feb 28, 2022

Codecov Report

Merging #1175 (c3dd924) into master (504aa1a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1175   +/-   ##
=======================================
  Coverage   78.32%   78.32%           
=======================================
  Files         271      271           
  Lines        7452     7452           
=======================================
  Hits         5837     5837           
  Misses       1280     1280           
  Partials      335      335           

@harkirat22 harkirat22 merged commit b909460 into tenable:master Mar 3, 2022
@harkirat22 harkirat22 deleted the feature/s3_bucket_policies_v4 branch March 3, 2022 00:57
@yu-iskw
Copy link
Contributor

yu-iskw commented Mar 3, 2022

@gaurav-gogia @harkirat22 It would be nice to revert the PR so that we quickly handle the issue, if I am correct.
#1178 .

The unit test of the added policies was failed.
https://github.com/accurics/terrascan/runs/5400638161?check_suite_focus=true .

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 AWS Terraform provider v4 for S3 buckets
3 participants