Skip to content

Commit

Permalink
fix(elasticloadbalancingv2): access logging fails on imported bucket (#…
Browse files Browse the repository at this point in the history
…27948)

#27558 fixed a circular dependency issue between the bucket and the ELB when using `logAccessLogs` by adding the dependency at the L1 level.
The dependency was added on both the bucket and the bucket policy (which already refers to the bucket).

This introduced a new bug when adding a dependency for imported buckets (as `bucket.node.defaultChild` is `undefined`).

This PR fixes the issue by adding the dependency at the L1 level on the bucket policy directly.

Closes #27928.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
lpizzinidev authored Nov 14, 2023
1 parent f2b4c62 commit ba11f36
Show file tree
Hide file tree
Showing 12 changed files with 2,359 additions and 21 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit ba11f36

Please sign in to comment.