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

update workflows #42

Merged
merged 1 commit into from
Jan 16, 2023
Merged

update workflows #42

merged 1 commit into from
Jan 16, 2023

Conversation

mamrajyadav
Copy link
Contributor

  • updated tfsec in github/workflows.

@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output
Result #1 MEDIUM VPC Flow Logs is not enabled for VPC  
────────────────────────────────────────────────────────────────────────────────
  main.tf:22-43
────────────────────────────────────────────────────────────────────────────────
   22  ┌ resource "aws_vpc" "default" {
   23count = var.vpc_enabled ? 1 : 0
   2425cidr_block                       = var.cidr_block
   26instance_tenancy                 = var.instance_tenancy
   27enable_dns_hostnames             = var.enable_dns_hostnames
   28enable_dns_support               = var.enable_dns_support
   29enable_classiclink               = var.enable_classiclink
   30enable_classiclink_dns_support   = var.enable_classiclink_dns_support
   ..  
────────────────────────────────────────────────────────────────────────────────
          ID aws-ec2-require-vpc-flow-logs-for-all-vpcs
      Impact Without VPC flow logs, you risk not having enough information about network traffic flow to investigate incidents or identify security issues.
  Resolution Enable flow logs for VPC

  More Information
  - https://aquasecurity.github.io/tfsec/latest/checks/aws/ec2/require-vpc-flow-logs-for-all-vpcs/
────────────────────────────────────────────────────────────────────────────────


  timings
  ──────────────────────────────────────────
  disk i/o             89.3µs
  parsing              86.306072ms
  adaptation           119.8µs
  checks               12.045252ms
  total                98.560424ms

  counts
  ──────────────────────────────────────────
  modules downloaded   0
  modules processed    1
  blocks processed     51
  files read           3

  results
  ──────────────────────────────────────────
  passed               2
  ignored              0
  critical             0
  high                 0
  medium               1
  low                  0

  2 passed, 1 potential problem(s) detected.

@themaniskshah themaniskshah merged commit fff55eb into master Jan 16, 2023
@delete-merged-branch delete-merged-branch bot deleted the tfsec-01 branch January 16, 2023 14:27
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.

None yet

4 participants