Description
Description
When using attach_lb_log_delivery_policy
, the full policy shown at https://docs.aws.amazon.com/elasticloadbalancing/latest/network/enable-access-logs.html is not used.
The conditions specifying aws:SourceAccount
and aws:SourceArn
are not included.
See
terraform-aws-s3-bucket/main.tf
Line 665 in 4c7f358
I see we can lock down a bit more using lb_log_delivery_policy_source_organizations
, however ideally, we should be able to lock down to a single or multiple accounts.
Is this intentional? Or is it a missing variable that could be added, e.g., lb_log_delivery_policy_source_accounts
?
- ✋ I have searched the open/closed issues and my issue is not listed.
Versions
-
Module version [Required]: latest
-
Terraform version: latest
-
Provider version(s): latest
Reproduction Code [Required]
See above
Expected behavior
We should be able to enforce only allowing logs from a single or multiple accounts.
Activity
github-actions commentedon May 25, 2025
This issue has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this issue will be closed in 10 days
stewartcampbell commentedon May 26, 2025
Not stale. I'm happy to contribute a PR but need guidance first from a maintainer on what the required approach would be.
elb_log_delivery_policy_source_organizations
variable #330feat: Add `elb_log_delivery_policy_source_organizations` variable
github-actions commentedon Jun 26, 2025
This issue has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this issue will be closed in 10 days
stewartcampbell commentedon Jun 26, 2025
@antonbabenko any feedback on this? I am happy to create PR but need some guidance from you guys.
antonbabenko commentedon Jun 26, 2025
@stewartcampbell Bryant has already left a review on PR #330. Please improve that PR.