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

feat: S3 생성 및 동영상 presigned url 조회 구현 #150

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

seok019283501
Copy link
Member

📋 상세 설명

  • AWS S3 기본 설정

📸 스크린샷

presigned url 조회
스크린샷 2024-11-21 오후 4 06 00

동영상 재생
스크린샷 2024-11-21 오후 4 07 29

@seok019283501 seok019283501 linked an issue Nov 21, 2024 that may be closed by this pull request
@github-actions github-actions bot added the feature 기능 추가 label Nov 21, 2024
Copy link
Contributor

@ywonchae1 ywonchae1 left a comment

Choose a reason for hiding this comment

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

presigned url 어려운데 수고 많으셨습니다 !!
제 풀리퀘 먼저 머지 하고 수정 한 번만 부탁드릴게요 🙇🏻‍♀️


@Tag(name = "Video Api", description = "Video 관련 API 목록입니다.")
public interface VideoApi {
@Operation(summary = "동영상의 presigned url을 발급받습니다..", description = "담당자: 최민석")
Copy link
Contributor

Choose a reason for hiding this comment

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

네.. ㅋㅋㅋㅋ ㅜㅜ

Comment on lines 19 to 24
s3:
access-key: ${AWS_S3_ACCESS_KEY}
secret-key: ${AWS_S3_SECRET_KEY}
bucket: ${AWS_S3_BUCKET}

Copy link
Contributor

Choose a reason for hiding this comment

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

요거 환경변수 객체로 분리해서 같이 수정해 주시면 좋을 거 같아요 !!


@Tag(name = "Video Api", description = "Video 관련 API 목록입니다.")
public interface VideoApi {
@Operation(summary = "동영상의 presigned url을 발급받습니다..", description = "담당자: 최민석")
Copy link
Member

Choose a reason for hiding this comment

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

잘하셨어요..

Copy link
Member Author

Choose a reason for hiding this comment

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

감사합니다..

Copy link
Contributor

@ywonchae1 ywonchae1 left a comment

Choose a reason for hiding this comment

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

따봉하나드립니다~

@seok019283501 seok019283501 merged commit e76f8f1 into develop Nov 21, 2024
1 check passed
@seok019283501 seok019283501 deleted the feature/148-s3 branch November 21, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐶 S3 연결
3 participants