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

Update variables.tf #19

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Update variables.tf #19

wants to merge 5 commits into from

Conversation

htutac
Copy link

@htutac htutac commented Feb 15, 2024

Added time_zone parameter for the rds parameter group to set that US/Eastern

Fixes Issue: link to JIRA ticket

Description:

  • Added time_zone var to rds parameter group to set that US/Eastern

Security Impact Analysis Questionnaire

Submitter Checklist

  • Is there an impact on Auditing and Logging procedures or capabilities?
  • Is there an impact on Authentication procedures or capabilities?
  • Is there an impact on Authorization procedures or capabilities?
  • Is there an impact on Communication Security procedures or capabilities?
  • Is there an impact on Cryptography procedures or capabilities?
  • Is there an impact on Sensitive Data procedures or capabilities?
  • Is there an impact on any other security-related procedures or capabilities?
  • No security impacts identified.

Security Risks Identified - For any applicable items on the "Submitter Checklist," describe the impact of the change and any implemented mitigations.

Added time_zone parameter for the rds parameter group to set that US/Eastern
@htutac htutac self-assigned this Feb 15, 2024
@robo-gotham
Copy link

robo-gotham commented Feb 15, 2024

Snyk Scanning for Commit: ea1583c

Snyk Infrastructure as Code

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

Issues
No vulnerable paths were found!


Test Summary

Organization: batcave-ispg
Project name: CMS-Enterprise/batcave-tf-mysql

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


Report Complete

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

Copy link
Contributor

@revaspeho revaspeho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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