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

fix xml output #291

Merged
merged 1 commit into from
Aug 18, 2020
Merged

fix xml output #291

merged 1 commit into from
Aug 18, 2020

Conversation

kanchwala-yusuf
Copy link
Contributor

Fixes #290

This fix creates an xml output:

<results>
  <violations>
    <violation rule_name="cloudfrontNoLogging" description="Ensure that your AWS Cloudfront distributions have the Logging feature enabled in order to track all viewer requests for the content delivered through the Content Delivery Network (CDN)." rule_id="AWS.CloudFront.Logging.Medium.0567" severity="MEDIUM" category="Logging" resource_name="s3-distribution-TLS-v1" resource_type="aws_cloudfront_distribution" file="aws_cloudfront_distribution.tf" line="7"></violation>
    <violation rule_name="cloudfrontNoSecureCiphers" description="Secure ciphers are not used in CloudFront distribution" rule_id="AWS.CloudFront.EncryptionandKeyManagement.High.0408" severity="HIGH" category="Encryption and Key Management" resource_name="s3-distribution-TLS-v1" resource_type="aws_cloudfront_distribution" file="aws_cloudfront_distribution.tf" line="7"></violation>
    <violation rule_name="cloudfrontNoHTTPSTraffic" description="Use encrypted connection between CloudFront and origin server" rule_id="AWS.CloudFront.EncryptionandKeyManagement.High.0407" severity="HIGH" category="Encryption and Key Management" resource_name="s3-distribution-TLS-v1" resource_type="aws_cloudfront_distribution" file="aws_cloudfront_distribution.tf" line="7"></violation>
    <violation rule_name="cloudfrontNoHTTPSTraffic" description="Use encrypted connection between CloudFront and origin server" rule_id="AWS.CloudFront.EncryptionandKeyManagement.High.0407" severity="HIGH" category="Encryption and Key Management" resource_name="s3-distribution-TLS-v1" resource_type="aws_cloudfront_distribution" file="aws_cloudfront_distribution.tf" line="7"></violation>
    <violation rule_name="cloudfrontNoGeoRestriction" description="Ensure that geo restriction is enabled for your Amazon CloudFront CDN distribution to whitelist or blacklist a country in order to allow or restrict users in specific locations from accessing web application content." rule_id="AWS.CloudFront.Network Security.Low.0568" severity="LOW" category="Network Security" resource_name="s3-distribution-TLS-v1" resource_type="aws_cloudfront_distribution" file="aws_cloudfront_distribution.tf" line="7"></violation>
  </violations>
  <count low="1" medium="1" high="3" total="5"></count>
</results>

@sonarcloud
Copy link

sonarcloud bot commented Aug 18, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Aug 18, 2020

Codecov Report

Merging #291 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #291   +/-   ##
=======================================
  Coverage   60.08%   60.08%           
=======================================
  Files          48       48           
  Lines         982      982           
=======================================
  Hits          590      590           
  Misses        344      344           
  Partials       48       48           

@williepaul williepaul merged commit c44a663 into master Aug 18, 2020
@williepaul williepaul deleted the bugfix/xml-output branch August 18, 2020 03:59
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.

Error with XML output
3 participants