Skip to content

feat: update secrets manager input and add s3 delete permission#33

Merged
alismx merged 1 commit intomainfrom
alis/to17/s3fix1
Jun 4, 2025
Merged

feat: update secrets manager input and add s3 delete permission#33
alismx merged 1 commit intomainfrom
alis/to17/s3fix1

Conversation

@alismx
Copy link
Copy Markdown
Collaborator

@alismx alismx commented Jun 4, 2025

Changes Proposed

  • Rename secrets_manager_postgresql_connection_string_version to secrets_manager_connection_string_version in documentation.
  • Add s3:DeleteObject action to the ecr_viewer_s3 data policy.

Testing

  • Verify that the new variable name is correctly documented and works as expected.
  • Check the IAM policy in AWS to ensure that the "s3:DeleteObject" permission is now allowed for the ECR viewer role.

- Rename secrets_manager_postgresql_connection_string_version to secrets_manager_connection_string_version in documentation.
- Add s3:DeleteObject action to the ecr_viewer_s3 data policy.
@alismx alismx merged commit 8693b5f into main Jun 4, 2025
2 checks passed
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.

2 participants