Skip to content

feat: read the secrets from Parameter Store instead of the environment - #231

Merged
lens0021 merged 2 commits into
mainfrom
claude/lambda-secrets-ssm
Oct 27, 2026
Merged

lens0021 merged 2 commits into
mainfrom
claude/lambda-secrets-ssm

Conversation

@lens0021

Copy link
Copy Markdown
Member

The deploy workflow writes each secret to a SecureString parameter under /lambda/<function>/, and the function's environment now holds only <NAME>_PARAMETER. A small secret() helper reads and caches the value, like bounce-handler already does with its token.

Needs femiwiki/infra#1569 applied first, which grants the reads and writes.


Generated by Claude Code

@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.50746% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.27%. Comparing base (6d121d7) to head (6a79aaf).

Files with missing lines Patch % Lines
sns-discord/lambda_function.py 85.71% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   96.05%   96.27%   +0.21%     
==========================================
  Files          10       10              
  Lines        1116     1180      +64     
==========================================
+ Hits         1072     1136      +64     
  Misses         44       44              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JnMfQTr1ciDwiu3fWfjMnL
@lens0021
lens0021 merged commit 6a4df7e into main Oct 27, 2026
12 checks passed
@lens0021
lens0021 deleted the claude/lambda-secrets-ssm branch October 27, 2026 16:51
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