Skip to content

feat: centralized secrets management (#289)#455

Merged
AnnabelJoe merged 1 commit into
AnnabelJoe:mainfrom
ladinoraa:issue-289-secrets-management
Jun 4, 2026
Merged

feat: centralized secrets management (#289)#455
AnnabelJoe merged 1 commit into
AnnabelJoe:mainfrom
ladinoraa:issue-289-secrets-management

Conversation

@ladinoraa

Copy link
Copy Markdown
Contributor

Summary

Closes #289

Establishes a consistent, centralized approach to secrets management across all environments.

Changes

  • apps/web/.env.example — expanded with all required variable names, inline docs, per-environment guidance in the header
  • .github/workflows/ci.yml — replaced hardcoded placeholder values with proper ${{ secrets.* }} references
  • docs/ONBOARDING.md — added callout box clarifying local-dev / CI / production secret boundaries
  • docs/SECRETS.md (new) — canonical reference: required secrets table, per-environment setup, Vercel production config, key rotation procedure, secret scanning notes

Acceptance criteria

  • All secrets stored in GitHub Actions secrets for CI/CD
  • Local development uses .env.local (gitignored)
  • Production secrets stored in Vercel environment variables
  • .env.example updated with all required variable names
  • No secrets committed to the repository

…oe#289)

Closes AnnabelJoe#289

- Documents all required secrets per environment (local / CI / production)
- Explains gitignored .env.local for local dev
- Lists all GitHub Actions secrets needed for CI and staging
- Documents Vercel env vars for production
- Covers production AWS Secrets Manager usage (no plaintext keys)
- Documents key rotation procedure
- References gitleaks / pre-commit secret scanning
@drips-wave

drips-wave Bot commented Jun 4, 2026

Copy link
Copy Markdown

@ladinoraa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@AnnabelJoe AnnabelJoe merged commit f01f302 into AnnabelJoe:main Jun 4, 2026
8 of 16 checks passed
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configure environment-specific secrets management

2 participants