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

Fix iac vul #366

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Fix iac vul #366

merged 1 commit into from
Sep 19, 2024

Conversation

Korjen97
Copy link
Collaborator

No description provided.

@AlvoBen
Copy link
Collaborator

AlvoBen commented Sep 19, 2024

Logo
Checkmarx One – Scan Summary & Details – bd595cd6-90bc-456a-a404-0d30a692878b

Fixed Issues

Severity Issue Source File / Package
HIGH Passwords And Secrets - Generic Password /release.yml: 77
MEDIUM ALB Deletion Protection Disabled /positive1.tf: 15
MEDIUM ALB Deletion Protection Disabled /positive1.tf: 15
MEDIUM ALB Listening on HTTP /positive1.tf: 9
MEDIUM ALB Listening on HTTP /positive1.tf: 9
MEDIUM ALB Not Dropping Invalid Headers /positive1.tf: 15
MEDIUM ALB Not Dropping Invalid Headers /positive1.tf: 15
LOW APT-GET Missing Flags To Avoid Manual Input /Dockerfile: 5
LOW APT-GET Missing Flags To Avoid Manual Input /Dockerfile: 5
LOW Healthcheck Instruction Missing /Dockerfile: 1
LOW Healthcheck Instruction Missing /Dockerfile: 1
LOW IAM Access Analyzer Not Enabled /positive1.tf: 1
LOW IAM Access Analyzer Not Enabled /positive1.tf: 1
LOW Shield Advanced Not In Use /positive1.tf: 15
LOW Shield Advanced Not In Use /positive1.tf: 15

@Korjen97 Korjen97 merged commit e65aa65 into main Sep 19, 2024
5 checks passed
Korjen97 added a commit that referenced this pull request Sep 24, 2024
@Korjen97
Revert "Fix iac vul (#366)"
7b7c350 
This reverts commit e65aa65.
@Korjen97 Korjen97 mentioned this pull request Sep 24, 2024
Merged
OrShamirCM pushed a commit that referenced this pull request Sep 24, 2024
@Korjen97
Revert "Fix iac vul (#366)" (#369)
79ce114 
This reverts commit e65aa65.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlvoBen AlvoBen AlvoBen approved these changes

@OrShamirCM OrShamirCM Awaiting requested review from OrShamirCM OrShamirCM is a code owner

@pedrompflopes pedrompflopes Awaiting requested review from pedrompflopes pedrompflopes is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Korjen97 @AlvoBen