(aws-lambda): fromDockerBuild output is located incorrectly

In v1.92.0, when fromDockerBuild is used, it is expected that contents /asset is copied, but instead the whole /asset is copied to output path, this makes Lambda handler reference index.handler failed, it has to be asset/index.handler.

Reproduction Steps

Create a index.js.

exports.handler = () => {
  console.info("Test");
}

A Dockerfile:

FROM amazon/aws-sam-cli-build-image-nodejs14.x
COPY index.js /asset/

Use lambda.Code.fromDockerBuild:

export class ExampleStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    new lambda.Function(this, "ExampleFunction", {
      runtime: lambda.Runtime.NODEJS_14_X,
      handler: "index.handler",
      code: lambda.Code.fromDockerBuild(ROOT_DIR),
    });
  }
}

What did you expect to happen?

The Lamda code contains only index.js.

What actually happened?

The Lambda code contains asset/index.js.

Environment

• CDK CLI Version : 1.92.0
• Framework Version: 1.92.0
• Node.js Version: 14
• OS : macOS Catalina
• Language (Version): TypeScript 4.2.3

Other

This is caused by expected behavior of docker cp.

Suggestion: automatically append /. to imagePath if it doesn’t.

---

This is 🐛 Bug Report

[jogold]

@phuctm97 are you taking this? I see a commit on your fork that references this issue.

I was thinking that something like this would do it:

public static fromDockerBuild(path: string, options: DockerBuildAssetOptions = {}): AssetCode {
  const imagePath = options.imagePath ?? '/asset';
  const assetPath = cdk.DockerImage
    .fromBuild(path, options)
    // ensure imagePath ends with /. to copy the **content** at this path
    .cp(imagePath.replace(/(\/|\/\.)?$/, '/.'), options.outputPath);
  return new AssetCode(assetPath);
}

I would personally not change the cp() method in core, I think it must remain general purpose.

(cc @eladb)

[github-actions[bot]]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.