You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Blast radius analysis requires existing infrastructure to examine. Once these resources are applied, future changes will show their dependencies.
🧠 Reasoning · ✖ 1 · ✔ 0
SNS production alert emails to external or unverified address: data leakage and alert delivery risk
Observations 1
Hypothesis
An SNS email subscription (protocol=email) is being created on the production-api-alerts topic to send alerts to alerts@example.com. If this email address is external, misconfigured, or not properly verified, production alert contents may be exposed outside the organization, leading to potential data exfiltration of sensitive operational information. Additionally, if this endpoint is incorrect or undeliverable, critical production alerts may fail to reach on-call recipients, degrading incident detection and response. The risk depends on the correctness and ownership of alerts@example.com and on SNS topic access controls governing who can publish to production-api-alerts.
Investigation
I loaded our monitoring and IAM standards; neither defines a blanket prohibition on SNS email subscriptions, so organizational policy does not in itself make this a risk. I inspected the blast radius state of the production topic; its resource policy uses the default constraint with Principal "*" but Condition AWS:SourceOwner = 540044833068, which restricts publish/manage actions to the owning account and does not expose the topic publicly. This addresses the hypothesis’ concern about permissive topic access controls.
The planned change adds a new SNS subscription with protocol email to alerts@example.com. By AWS design, email subscriptions receive no messages until the recipient explicitly confirms via the link sent by SNS; unconfirmed subscriptions remain in PendingConfirmation and do not deliver notifications. Therefore, no alert content will be sent anywhere unless that mailbox owner confirms. (docs.aws.amazon.com)
Regarding apply behavior: Terraform’s aws_sns_topic_subscription does not wait for confirmation for the email protocol, and confirmation_timeout_in_minutes only applies to HTTP/HTTPS endpoints. So this change will not block on confirmation; it will create a pending subscription that is inert unless manually confirmed. Unconfirmed subscriptions are automatically deleted by SNS after 48 hours, further limiting lingering exposure. (man.hubwiz.com)
Given the above, there is no concrete evidence that sensitive production alerts will be delivered to an external party as a result of this change, nor that alerting efficacy will degrade (this is an additional subscription, not a replacement). Any leakage would require an out‑of‑band confirmation by whoever controls alerts@example.com, which is not part of the planned apply. Therefore, the stated risk is speculative and not a real, actionable risk tied to this change.
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
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.
This PR contains the following updates:
🔧 This Pull Request updates lock files to use the latest dependency versions.
Configuration
📅 Schedule: Branch creation - "before 4am on monday" in timezone Europe/London, Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.