Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Putting the documentation link into refactored code base #8

Merged
merged 1 commit into from
May 1, 2024

Conversation

JamesEElliott
Copy link

No description provided.

@robo-gotham
Copy link

Snyk Scanning for Commit: c9ae307

Snyk Infrastructure as Code

  • Snyk testing Infrastructure as Code configuration issues.
    ✔ Test completed.

Issues

Medium Severity Issues: 1

[Medium] Wildcard principal in KMS key access policy
Info: Wildcard principal has been specified in access policy. Using wild
card will grant unnecessary access to any user in the account
Rule: https://security.snyk.io/rules/cloud/SNYK-CC-AWS-709
Path: [DocId: 0] > Resources[FileAssetsBucketEncryptionKey] > Properties >
KeyPolicy
File: bootstrap/bootstrap-template.yaml
Resolve: Set Principal attribute in the policy to specific entities, for
example arn:aws:iam::123456789012:user/JohnDoe

High Severity Issues: 1

[High] Wildcard action in IAM Policy
Info: The IAM policy allows all actions on resource. Granting permission to
perform any action is against 'least privilege' principle
Rule: https://security.snyk.io/rules/cloud/SNYK-CC-TF-69
Path: [DocId: 0] > Resources[CdkBoostrapPermissionsBoundaryPolicy] >
Properties > PolicyDocument
File: bootstrap/bootstrap-template.yaml
Resolve: Set Action attribute in Properties.PolicyDocument to specific
actions e.g. s3:ListBucket


Test Summary

Organization: batcave-ispg
Project name: CMS-Enterprise/batcave-wt-sdl-reporting

✔ Files without issues: 0
✗ Files with issues: 1
Ignored issues: 0
Total issues: 2 [ 0 critical, 1 high, 1 medium, 0 low ]


Report Complete

Your test results are available at: https://snyk.io/org/batcave-ispg/projects
under the name: CMS-Enterprise/batcave-wt-sdl-reporting

@JamesEElliott JamesEElliott requested a review from rileydakota May 1, 2024 14:56
@rileydakota
Copy link
Contributor

LGTM

@rileydakota rileydakota merged commit 5ce92bb into main May 1, 2024
3 of 4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants