what
Change the event_selector var from a map to a list type.
why
It is currently a type map, that then gets put inside a list.
Even though it is a null map by default, because it is embedded into a list, it is evaluated by the cloudtrail resource.
And because it gets evaluated by the resource it triggers this bug:
hashicorp/terraform-provider-aws#5448
By switching it to a list, it means that by default instead of being a mull map in a null list, it is just a null list. Which is skipped over, and doesn't trigger the bug.
TF Apply with the Cloudtrail module always comes up with :
~ module.cloudtrail.aws_cloudtrail.default
event_selector.#: "0" => "1"
event_selector.0.include_management_events: "" => "true"
event_selector.0.read_write_type: "" => "All"