Skip to content

fix: update IAM key rotation period from 90 to 45 days#76

Open
mwirikia wants to merge 1 commit intomainfrom
KEH-2285_key_rotation_period
Open

fix: update IAM key rotation period from 90 to 45 days#76
mwirikia wants to merge 1 commit intomainfrom
KEH-2285_key_rotation_period

Conversation

@mwirikia
Copy link
Copy Markdown
Contributor

@mwirikia mwirikia commented May 6, 2026

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

What

Update key rotation from 90d to 45d.

Testing

Have any new tests been added as part of this issue? If not, try to explain why test coverage is not needed here.

  • Yes
  • No
    Please write a brief description of why test coverage is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Documentation

Has any new documentation been written as part of this issue? We should try to keep documentation up to date
as new code is added, rather than leaving it for the future.

  • Yes
  • No
    Please write a brief description of why documentation is not necessary here.
  • Not as part of this ticket. (Could be done at a later point)

Related issues

Provide links to any related issues.

How to review

Describe the steps required to test the changes.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Warnings Elapsed time
⚠️ BASH bash-exec 6 4 0 0.02s
✅ BASH shellcheck 6 0 0 0.1s
✅ BASH shfmt 6 0 0 0.01s
✅ CSHARP csharpier 1 0 0 0.55s
✅ DOCKERFILE hadolint 1 0 0 0.11s
✅ JSON jsonlint 2 0 0 0.22s
✅ JSON prettier 2 0 0 0.37s
✅ JSON v8r 2 0 0 1.93s
✅ MARKDOWN markdownlint 6 0 0 0.35s
✅ REPOSITORY checkov yes no no 30.19s
✅ REPOSITORY dustilock yes no no 0.01s
✅ REPOSITORY gitleaks yes no no 3.57s
✅ REPOSITORY grype yes no no 48.59s
✅ REPOSITORY kics yes no no 32.5s
✅ REPOSITORY secretlint yes no no 0.87s
✅ REPOSITORY syft yes no no 2.52s
❌ REPOSITORY trivy yes 1 no 19.69s
✅ REPOSITORY trivy-sbom yes no no 0.29s
✅ TERRAFORM terraform-fmt 7 0 0 0.3s
✅ YAML prettier 8 0 0 0.65s
✅ YAML v8r 8 0 0 5.56s
✅ YAML yamllint 8 0 0 0.38s

See detailed report in MegaLinter reports

You could have same capabilities but better runtime performances if you request a new MegaLinter flavor.

MegaLinter is graciously provided by OX Security

Copy link
Copy Markdown

@co-opsw co-opsw 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 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.

3 participants