-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature(#59, #161) 발표자 미디어 파일 Object Storage에 저장, DB에 강의 정보 업데이트 #199
Feature(#59, #161) 발표자 미디어 파일 Object Storage에 저장, DB에 강의 정보 업데이트 #199
Conversation
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오늘도 고생하셨습니다 !
mediaServer/src/utils/ncp-storage.ts
Outdated
accessKeyId: access_key, | ||
secretAccessKey: secret_key | ||
} | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기까지 config로 빼는 건 어떠신가요?
mediaServer/src/utils/ncp-storage.ts
Outdated
} | ||
}); | ||
|
||
const saveFile = async (file: any, filename: string) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어디에 저장하는지에 대한 의미가 함수 이름에 있으면 좋을 것 같다는 생각이 있습니다 !
saveInStorage? 적절한 이름은 잘 모르겠네요..ㅎㅎ
리뷰 감사합니다! 말씀해주신거 반영했습니다 시간 되실 때 천천히 봐주세요!! 오늘 고생 많으셨습니다~ |
작업 개요
close #59
추출한 발표자의 미디어 파일을 NCP Object Storage에 저장한다.
close #161
강의가 종료되면 미디어서버에서 API 서버로 강의 정보와 함께 강의 완료 요청을 보낸다.
작업 사항
기존에 로컬에 저장되던 mp4 파일을 읽어와서 NCP Object Storage에 저장했습니다. 저장한 뒤에는 저장 위치 url을 반환합니다. 해당 url을 DB에 업데이트 해주기 위해서 미디어서버에서 API 서버로 요청을 보냅니다.
고민한 점들(필수 X)
NCP에서 AWS S3에서 제공하는 Javascript용 SDK를 이용해 Object Storage를 사용하는 API를 제공하기에 이를 사용했습니다. 그 이유는 나중에 S3로 바꾸게 된다면 코드는 바꿀 필요 없이 환경변수 값만 바꿔주면 되기 때문에 확장성 부분에서 좋을 것 같다고 생각했기 때문입니다. 코드로 구현하는 과정에서도 S3를 사용해봤던 입장으로 더 익숙하기에 이를 사용했습니다.