Skip to content

Commit

Permalink
docs(alb): ALB requires region to use logging (#5085)
Browse files Browse the repository at this point in the history
* Add comment about requiring region to use logging

* Update application-load-balancer.ts

Co-authored-by: Rico Huijbers <rix0rrr@gmail.com>
  • Loading branch information
2 people authored and mergify[bot] committed Dec 20, 2019
1 parent b700b77 commit 5a4874b
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,10 @@ export class ApplicationLoadBalancer extends BaseLoadBalancer implements IApplic
}

/**
* Enable access logging for this load balancer
* Enable access logging for this load balancer.
*
* A region must be specified on the stack containing the load balancer; you cannot enable logging on
* environment-agnostic stacks. See https://docs.aws.amazon.com/cdk/latest/guide/environments.html
*/
public logAccessLogs(bucket: s3.IBucket, prefix?: string) {
this.setAttribute('access_logs.s3.enabled', 'true');
Expand Down

0 comments on commit 5a4874b

Please sign in to comment.