Skip to content

fix: update ExcludeCharacters in elasticache.yaml for password generation - #21

Merged
guimoreirar merged 1 commit into
mainfrom
fix/elasticache-secret-string
Mar 25, 2026
Merged

fix: update ExcludeCharacters in elasticache.yaml for password generation#21
guimoreirar merged 1 commit into
mainfrom
fix/elasticache-secret-string

Conversation

@guimoreirar

Copy link
Copy Markdown
Member
  • Added '{' and '}' to the ExcludeCharacters list in the GenerateSecretString section to enhance security for generated passwords.

…tion

- Added '{' and '}' to the ExcludeCharacters list in the GenerateSecretString section to enhance security for generated passwords.
@guimoreirar guimoreirar self-assigned this Mar 25, 2026
@coderabbitai

coderabbitai Bot commented Mar 25, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8fa4dac0-8150-419a-aa2d-c33a99d4ad8e

📥 Commits

Reviewing files that changed from the base of the PR and between 5502d4e and 6a7ab00.

📒 Files selected for processing (1)
  • templates/elasticache.yaml

Walkthrough

The ElastiCache authentication token secret configuration in templates/elasticache.yaml was updated to exclude an additional set of characters when generating the secret. Specifically, the ExcludeCharacters parameter in the GenerateSecretString configuration was modified to add curly braces ({}) to the list of disallowed characters. This changes the character set available for CloudFormation's automatic 64-character secret generation used by the ElastiCache replication group's auth token.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating ExcludeCharacters in elasticache.yaml for password generation, which aligns with the changeset.
Description check ✅ Passed The description is directly related to the changeset, specifying that curly braces were added to ExcludeCharacters for security enhancement.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@lerian-studio

Copy link
Copy Markdown

Consider updating CHANGELOG.md to document this change. If this change doesn't need a changelog entry, add the skip-changelog label.

@guimoreirar
guimoreirar merged commit 7bdb049 into main Mar 25, 2026
25 checks passed
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