Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3주차] 도전 과제 제출 #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[3주차] 도전 과제 제출 #3

wants to merge 2 commits into from

Conversation

HoYoungChun
Copy link
Contributor

🐕 과제 구현 명세

Netflix API 추가 후에 배포 진행했습니다!
동적이지 않은 매우 정적인 상태인데, 추후 디벨롭하겠습니다.

스크린샷 2022-11-08 오후 10 00 51

구현한 엔드포인트

http://43.200.22.42:3000/api/video/3

스크린샷 2022-11-08 오후 10 01 57

🐥 이런 점이 새로웠어요 / 어려웠어요

아직은 node 생태계가 어색한 것 같습니다. 오히려 코드 구현은 괜찮은데, 그 이전의 세팅과정이 어색해서 빨리 익숙해져야 겠다는 생각을 했습니다.

PM2라는 친구가 매우 매력적이었습니다! 아마 프로덕션에 배포할때 dockerize를 할 것 같은데 그때 잘 이용하면 시너지가 좋을 것 같아요.

S3 관련해서 완전 Public으로 두면 링크가 한번 노출되었을 때 READ매크로 대응에 힘들 것 같아 프로덕션에서는 presigned url을 이용해야겠다고 생각했습니다.

@HoYoungChun HoYoungChun added the assignment Assignment for a week label Nov 8, 2022
@HoYoungChun HoYoungChun self-assigned this Nov 8, 2022
@devkwonsehoon
Copy link
Contributor

Presigned URL 도 좋은 대안입니다!
AWS SDK 이용한 helper를 두는 것도 괜찮을 것 같군요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assignment Assignment for a week
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants