Skip to content

[5주차] 워크북 미션-roy#10

Open
chans20 wants to merge 5 commits intomainfrom
feat/week5-roy
Open

[5주차] 워크북 미션-roy#10
chans20 wants to merge 5 commits intomainfrom
feat/week5-roy

Conversation

@chans20
Copy link
Copy Markdown
Collaborator

@chans20 chans20 commented May 1, 2026

.

@drghdtjr
Copy link
Copy Markdown
Contributor

drghdtjr commented May 1, 2026

전체적으로 미션 수행하시느라 고생 많으셨습니다!
멀티파트 데이터를 처리하기 위해 @Setter를 남용하지 않고, @ModelAttribute를 사용하여 객체 형태로 한 번에 데이터를 바인딩하려는 시도가 인상적입니다.
향후 S3와 같은 외부 스토리지 확장성을 고려했을 때, Presigned URL 방식을 도입해 보는 것도 함께 고민해 보시면 좋을 것 같습니다. 현재처럼 서버가 파일을 직접 받아 전달하는 방식 대신, 서버는 권한이 담긴 URL만 생성해 주고 클라이언트가 스토리지로 직접 업로드하게 하면 서버의 부하를 획기적으로 줄일 수 있고 보안성도 높일 수 있습니다.
수고하셨습니다.

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.

2 participants