Skip to content

Commit

Permalink
use serverAccessLogs and reference the L1 construct directly through …
Browse files Browse the repository at this point in the history
…LazyValue
  • Loading branch information
Solis Ramirez committed Dec 2, 2019
1 parent b91b82c commit 2bd9431
Showing 1 changed file with 0 additions and 20 deletions.
20 changes: 0 additions & 20 deletions packages/@aws-cdk/aws-s3/lib/bucket.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,13 +75,6 @@ export interface IBucket extends IResource {
*/
addToResourcePolicy(permission: iam.PolicyStatement): void;

/**
* Adds the provided bucket as a destination bucket for this bucket server access logs.
* @param destinationBucket Bucket in which the server access logs will be saved in.
* @param optional logFilePrefix of the access logs.
*/
enableServerAccessLogs(destinationBucket: IBucket, logFilePrefix?: string): void;

/**
* The https URL of an S3 object. For example:
* @example https://s3.us-west-1.amazonaws.com/onlybucket
Expand Down Expand Up @@ -352,19 +345,6 @@ abstract class BucketBase extends Resource implements IBucket {
return rule;
}

/**
* Sets the passed bucket as destination bucket for this bucket's access logs.
*
* @param destinationBucket The bucket where the access logs will be stored.
* @param logFilePrefix Optional log file prefix.
*/
public enableServerAccessLogs(destinationBucket: IBucket, logFilePrefix?: string): void {
(this.node.defaultChild as CfnBucket).loggingConfiguration = {
destinationBucketName: destinationBucket.bucketName,
logFilePrefix,
};
}

/**
* Defines an AWS CloudWatch event that triggers when an object is uploaded
* to the specified paths (keys) in this bucket using the PutObject API call.
Expand Down

0 comments on commit 2bd9431

Please sign in to comment.