python docs are incoreect for aws_s3.CfnBucket "logging_configuration"

Hi,

I’m trying to set the S3 Access Logging using the aws_s3.CfnBucket resource.

Based on the documentation found here and here, I’m adding the value { "DestinationBucketName" : "MyLoggingBucket", "LogFilePrefix" : "logging-"} to the logging_configuration parameter.

The problem is that when I generate the CloudFormation Template, I see always an empty LoggingConfiguration property, as the example below:

Resources:
  bucket:
    Type: AWS::S3::Bucket
    Properties:
      BucketName: my_bucket
      LoggingConfiguration: {}
    Metadata:
      aws:cdk:path: logging/bucket
  CDKMetadata:
    Type: AWS::CDK::Metadata
    Properties:
      Modules: aws-cdk=1.15.0,@aws-cdk/assets=1.15.0,@aws-cdk/aws-cloudwatch=1.15.0,@aws-cdk/aws-ec2=1.15.0,@aws-cdk/aws-events=1.15.0,@aws-cdk/aws-iam=1.15.0,@aws-cdk/aws-kms=1.15.0,@aws-cdk/aws-lambda=1.15.0,@aws
-cdk/aws-logs=1.15.0,@aws-cdk/aws-s3=1.15.0,@aws-cdk/aws-s3-assets=1.15.0,@aws-cdk/aws-s3-notifications=1.15.0,@aws-cdk/aws-sns=1.15.0,@aws-cdk/aws-sqs=1.15.0,@aws-cdk/aws-ssm=1.15.0,@aws-cdk/core=1.15.0,@aws-cdk/c
x-api=1.15.0,@aws-cdk/region-info=1.15.0,jsii-runtime=Python/3.7.5

I think this is a bug (at least in Python) that blocks me into getting further case.

Reproduction Steps

Create a new Stack using Python and add the following code snippet there:

bucket = s3.CfnBucket(
    self, "bucket",
    bucket_name="my_bucket",
    logging_configuration={
        "DestinationBucketName": "MyLoggingBucket",
        "LogFilePrefix": "logging-",
    }
)

Then run cdk synth to see the generated CloudFormation Template.

Error Log

N/A

Environment

• CLI Version : 1.15.0
• Framework Version: 1.15.0
• OS : MacOS X Mojave
• Language : Python/3.7.5

Other

---

This is 🐛 Bug Report

[eduardohki]

Thanks for your reply @RomainMuller

I understand about the limitation in the Python implementation, and my wish is to use CfnBucket.LoggingConfigurationProperty (I did so).

The issue here IMO is the lack of this proper information in the Python Docs, as you stated.

[RomainMuller]

@eduardohki absolutely!