From c6da7448a8c5810b88eb511984d46f6338d48aac Mon Sep 17 00:00:00 2001 From: Perry Zhu Date: Sat, 1 Nov 2025 16:03:59 -0700 Subject: [PATCH] fix: pass env var --- scripts/Dockerfile.script | 3 +-- scripts/src/lambda-handler.ts | 6 +++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/scripts/Dockerfile.script b/scripts/Dockerfile.script index 7d31dc5..231e3c4 100644 --- a/scripts/Dockerfile.script +++ b/scripts/Dockerfile.script @@ -8,8 +8,6 @@ RUN npm run build FROM node:22 AS runtime WORKDIR /app -ARG AWS_S3_BUCKET_NAME - COPY --from=build /app/dist ./dist COPY package*.json ./ RUN apt-get update && \ @@ -23,6 +21,7 @@ RUN apt-get update && \ RUN npm install aws-lambda-ric RUN npm ci --only=production ENV PLAYWRIGHT_BROWSERS_PATH=/ms-playwright +ENV S3_BUCKET_NAME=${AWS_S3_BUCKET_NAME} RUN npx playwright install chromium --with-deps && \ npx playwright install-deps chromium diff --git a/scripts/src/lambda-handler.ts b/scripts/src/lambda-handler.ts index d603e9e..b89971b 100644 --- a/scripts/src/lambda-handler.ts +++ b/scripts/src/lambda-handler.ts @@ -34,14 +34,14 @@ export async function handler( const bucketName = process.env.S3_BUCKET_NAME; const key = "data/rolling-images.json"; - if (!bucketName || !key) { - console.error("Missing required environment variables: S3_BUCKET_NAME or S3_OBJECT_KEY"); + if (!bucketName) { + console.error("Missing required environment variable: S3_BUCKET_NAME"); return { statusCode: 500, body: JSON.stringify({ success: false, error: "Missing S3 configuration", - message: "S3_BUCKET_NAME and S3_OBJECT_KEY environment variables are required", + message: "S3_BUCKET_NAME environment variable is required", image_links: imageLinks, total_count: imageLinks.length, }),