feat(ecs_patterns): Enable enableExecutecommand as default

[deuscapturus]

Expose enableExecutecommand as a property in ecs_patterns constructs for Fargate and Ec2.

Default enableExecutecommand = true.

Enabling execute command is a sensible default for the high level ecs_patterns construct.

---

All Submissions:

• Have you followed the guidelines in our Contributing guide?

Adding new Unconventional Dependencies:

• This PR adds new unconventional dependencies following the process described here

New Features

• Have you added the new feature to an integration test?
  • Did you use yarn integ to deploy the infrastructure and generate the snapshot (i.e. yarn integ without --dry-run)?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[gitpod-io]

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
• Commit ID: 6acab90
• Result: FAILED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[hoegertn]

My 2ct: While I love being able to activate this feature, I do not think it should be active by default.

Being able to execute into a container is not a safe default from a security and/or compliance point of view.

[LukvonStrom]

Integrating without setting as default is done at #18663 @hoegertn

[deuscapturus]

I'll close if #18663 is merged.

[github-actions]

This PR has been in the BUILD FAILING state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

[github-actions]

This PR has been deemed to be abandoned, and will be automatically closed. Please create a new PR for these changes if you think this decision has been made in error.