Skip to content

feat: Restrict IAM Lambda secret rotation role (DBTP-2964)#1420

Draft
BenFielder wants to merge 10 commits intomainfrom
lambda-iam-role
Draft

feat: Restrict IAM Lambda secret rotation role (DBTP-2964)#1420
BenFielder wants to merge 10 commits intomainfrom
lambda-iam-role

Conversation

@BenFielder
Copy link
Copy Markdown

Addresses https://uktrade.atlassian.net/browse/DBTP-.


Checklist:

Title:

Description:

  • Link to ticket included (unless it's a quick out of ticket thing)
  • Includes tests (or an explanation for why it doesn't)
  • If the work includes user interface changes, before and after screenshots included in description
  • Includes any applicable changes to the documentation in this code base
  • Includes link(s) to any applicable changes to the documentation in the DBT Platform Documentation (can be to a pull request)

Tasks:

Reviewer Checklist

  • I have reviewed the PR and ensured no secret values are present

@BenFielder BenFielder requested a review from a team as a code owner April 30, 2026 13:24
@BenFielder BenFielder marked this pull request as draft April 30, 2026 13:24
@github-actions
Copy link
Copy Markdown
Contributor

Your PR has commits that are missing the Signed-off-by trailer. This is likely due to the pre-commit hook not being configured on your local machine. The usual fix for this issue is to run pre-commit install --install-hooks --overwrite -t commit-msg -t pre-commit, however for more detailed help in setting up the pre-commit hooks, follow the instructions at https://github.com/uktrade/github-standards/blob/main/README.md#usage

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None

@BenFielder BenFielder marked this pull request as ready for review May 5, 2026 14:02
Comment thread terraform/application-load-balancer/main.tf Outdated
Comment thread terraform/application-load-balancer/main.tf Outdated
@kierdavis kierdavis self-requested a review May 6, 2026 14:56
@kierdavis kierdavis marked this pull request as draft May 6, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants