Skip to content

feat: add multipart file upload#238

Open
yuchem2 wants to merge 4 commits into
developfrom
CARGO-416
Open

feat: add multipart file upload#238
yuchem2 wants to merge 4 commits into
developfrom
CARGO-416

Conversation

@yuchem2

@yuchem2 yuchem2 commented Jul 1, 2026

Copy link
Copy Markdown
Member

source 데코레이터에 @File@Files를 추가하여 multipart/form-data로 들어오는 파일 데이터를 지원합니다. express 환경에서 제일 많이 사용되는 파서는 multer이기 때문에 multer를 기본으로 잡고 사용자의 요구에 따라 커스텀 파서를 추가할 수 있도록 코드를 작성합니다.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

이번 풀 리퀘스트는 express-cargo 패키지에 @File@Files 데코레이터를 추가하여 파일 업로드 바인딩 기능을 구현하고, 예제 애플리케이션 및 테스트 코드를 작성했습니다. 리뷰어는 @File@Files 필드에서 @Transform 데코레이터가 정상적으로 동작할 수 있도록 bindFile 함수 내에서 직접 값을 할당하는 대신 transformSource를 호출하도록 구조를 개선할 것을 제안했습니다. 또한, multerFileLocator에서 파일 객체가 배열이 아닌 경우를 대비해 방어적으로 배열 여부를 확인하고 처리하도록 코드를 보완할 것을 권장했습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread packages/express-cargo/src/binding.ts
Comment thread packages/express-cargo/src/binding.ts
Comment thread packages/express-cargo/src/fileHandler.ts
@yuchem2 yuchem2 requested a review from laggu July 1, 2026 14:12
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.

1 participant