Repository navigation
Conversation
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
tryindus.caSES identity.prepare-emailrollout stage that verifies the domain and DNS while Cognito retains its default sender. Refuse the sender switch until SES production access, DKIM, and MAIL FROM are verified. Keep the default-sender rollback path available.Verification
terraform fmt -check -recursive infra/optimized/terraformterraform validatein both optimized Terraform rootsterraform testin the optimized environment root (5 passed)actionlintnode --test scripts/capacity/*.test.mjs(11 passed)bash scripts/optimized/validate-isolation.shdocker compose --env-file infra/optimized/runtime/release.env.example -f infra/optimized/runtime/compose.yaml configbash -n scripts/optimized/install-release.sh infra/optimized/runtime/bootstrap.shgit diff --checkNo local E2E was run, as requested. The Caddy container validation from optimized verification could not run because the local Docker daemon is unavailable. Bootstrap provider installation ran out of disk space; validation passed using the already downloaded provider from the environment root.
The PR's optimized infrastructure and AWS platform jobs passed. The unrelated core application job fails its legacy Next.js dependency audit on two moderate advisories.
Deployment and migration impact
No database migration. This PR does not apply infrastructure. AWS requested further information and a verified SES identity for production access;
aws-alerts@tryindus.cahas been verified as a temporary account prerequisite. Before rollout, update the optimized bootstrap Terraform role, runprepare-email, and confirm the domain authentication and SES production access inus-east-1. Both rollout stages require a reviewed saved plan and production approval. The switch changes the sender for signup confirmation and password recovery; existing users are unchanged.Rollback
Set
enable_branded_cognito_email = falseinproduction-email.auto.tfvars, review the plan, and apply through the optimized infrastructure workflow. Cognito returns to its default sender. The SES production-access guard only applies to plans that keep the SES sender enabled.Deferred work
aws-alerts@tryindus.caSES email identity after the domain identity is verified and production access is approved.