Skip to content

Commit

Permalink
Do not rename asset/main
Browse files Browse the repository at this point in the history
The pull request I copied to create this pull request renamed
the handler, however the Go source is still named main.go, so
we should keep the main naming.
  • Loading branch information
SamStephens committed Nov 21, 2023
1 parent 4c19505 commit 6a53182
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions lambda/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,6 @@ RUN go env
COPY . /ws

RUN mkdir -p /asset/ && \
make OUTPUT=/asset/bootstrap && \
file /asset/bootstrap && \
ls -lh /asset/bootstrap
make OUTPUT=/asset/main && \
file /asset/main && \
ls -lh /asset/main
2 changes: 1 addition & 1 deletion src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ export class ECRDeployment extends Construct {
uuid: this.renderSingletonUuid(memoryLimit),
code: getCode(props.buildImage ?? 'public.ecr.aws/sam/build-go1.x:latest'),
runtime: lambda.Runtime.PROVIDED_AL2,
handler: 'bootstrap',
handler: 'main',
environment: props.environment,
lambdaPurpose: 'Custom::CDKECRDeployment',
timeout: Duration.minutes(15),
Expand Down

0 comments on commit 6a53182

Please sign in to comment.