Skip to content

fix: 이미지 처리 Lambda 모듈 로딩 오류 수정#33

Merged
lsy1307 merged 1 commit intomainfrom
32-fix-lambda-image-processing
May 6, 2026
Merged

fix: 이미지 처리 Lambda 모듈 로딩 오류 수정#33
lsy1307 merged 1 commit intomainfrom
32-fix-lambda-image-processing

Conversation

@lsy1307
Copy link
Copy Markdown
Contributor

@lsy1307 lsy1307 commented May 6, 2026

관련 이슈

작업 내용

  • nodejs22.x Lambda 런타임에서 index.js가 CommonJS로 로드되며 import 문법 때문에 실패하던 오류를 수정했습니다.
  • resizing Lambda와 thumbnail Lambda 모두 import/exportrequire/exports.handler로 변경했습니다.
  • S3 이벤트가 아닌 콘솔 테스트 이벤트처럼 Records가 없는 경우에도 Lambda가 예외 없이 skip하도록 보완했습니다.
  • S3 이벤트에 여러 record가 들어오는 경우 모든 record를 순차 처리하도록 변경했습니다.
  • thumbnail Lambda는 실제로 JPEG로 변환하므로 결과 파일 확장자를 .jpg로 저장하도록 맞췄습니다.

특이 사항

리뷰 요구사항 (선택)

- 상세내용: nodejs22.x 런타임에서 index.js가 CommonJS로 로드되도록 import/export 문법을 require/exports로 변경

- 상세내용: resizing과 thumbnail Lambda 모두 S3 이벤트 record 검증과 다중 record 처리를 추가

- 상세내용: thumbnail 결과 확장자를 실제 JPEG 변환 결과와 맞춰 .jpg로 저장
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 6, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ded5d5c0-b323-471d-a3de-3633df860298

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 32-fix-lambda-image-processing

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Terraform Plan: global

Plan: 0 to add, 2 to change, 0 to destroy.

전체 plan 결과는 보안을 위해 댓글에 포함되지 않습니다. 워크플로우 실행 아티팩트를 확인하세요.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

@coderabbitai review

Copy link
Copy Markdown
Contributor

@Hexeong Hexeong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lsy1307 lsy1307 merged commit 59f5f25 into main May 6, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 이미지 처리 Lambda 오류 수정

2 participants