Skip to content

VED-1163: Elasticache Security Configurations#1451

Open
Thomas-Boyle wants to merge 1 commit intomasterfrom
ved-1163-elasticache-security-config
Open

VED-1163: Elasticache Security Configurations#1451
Thomas-Boyle wants to merge 1 commit intomasterfrom
ved-1163-elasticache-security-config

Conversation

@Thomas-Boyle
Copy link
Copy Markdown
Contributor

PR Description

  • Added support for Redis authentication using AWS Secrets Manager.
  • Introduced a new random_password resource for generating Redis auth tokens.
  • Updated the aws_elasticache_replication_group to include authentication settings.
  • Modified environment variables in Lambda functions to include Redis connection details and auth token.
  • Enhanced the Redis client to retrieve the auth token from Secrets Manager.
  • Updated IAM policies to allow access to the Redis auth token.

This change improves security by managing Redis credentials through AWS Secrets Manager.

How were the changes tested

Screenshots to be provided

- Added support for Redis authentication using AWS Secrets Manager.
- Introduced a new `random_password` resource for generating Redis auth tokens.
- Updated the `aws_elasticache_replication_group` to include authentication settings.
- Modified environment variables in Lambda functions to include Redis connection details and auth token.
- Enhanced the Redis client to retrieve the auth token from Secrets Manager.
- Updated IAM policies to allow access to the Redis auth token.

This change improves security by managing Redis credentials through AWS Secrets Manager.
@github-actions
Copy link
Copy Markdown
Contributor

This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket:

VED-1163

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This PR contains a fix for existing functionality. terraform Pull requests that update terraform code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant