(aws-route53): add logging configuration method

As far as I know, the domain registration in AWS needs to be a manual step via the UI. That’s fine, however we would like then to make sure via CDK that the logging is properly configured for the HostedZone, but the HostedZone only allows the queryLogsLogArn to be done inside a constructor - so we have a chicken-egg problem.

Could a method for configuring the loggingGroup be added for a HostedZone retrieved by fromLookup ?

[njlynch]

The RFC you referenced is quite empty I’d say and I don’t think it contributes anyhow to getting visibility for this specific request.

It’s fair that the RFC doesn’t specifically call out Route53 HostedZone QueryLoggingConfig; however, I’d much rather have one issue to track the overall desire to be able to mutate imported resources than one per resource/property combination. It’s much harder for us to track impact of hundreds of feature requests with little/no interaction than a single RFC with tens/hundreds of 👍s. The foundational work to allow users to import existing resources into a CDK-owned Stack, and then be able to adjust properties of those resources, is going to be the same for Route53 Hosted Zones, Athena Work Groups, ECS Clusters, etc. Once we’ve done it for one, all should be supported (or easily incrementally added).

Furthermore, it’s not entirely true that CDK doesn’t support importing resources

Correct. What we don’t support is modifying imported resources, as I said above. You can import a HostedZone today; you just can’t change its QueryLoggingConfig. Similarly, you can import a Bucket today, but you can’t adjust the bucket encryption.

[rantoniuk]

Thanks for detailed explanation. The only comment I have is that… I don’t think that RFC 👍 works - it’s there for 2 years and only 1 👍🏻 - I’m sure that there are a lot of other use cases needing this 😃