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

[서지원] 5주차 과제 -complete #42

Merged
merged 3 commits into from
Nov 18, 2023
Merged

[서지원] 5주차 과제 -complete #42

merged 3 commits into from
Nov 18, 2023

Conversation

seojeewon
Copy link
Contributor

제출 상태

5주차 과제 complete

help

200 OK

Copy link
Member

@JeonHaeseung JeonHaeseung left a comment

Choose a reason for hiding this comment

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

과제 리뷰 완료
항상 열정적으로 과제해주시는 지원님!
문제 해결 방법도 따로 마크다운으로 올리셔서 읽기 편했답니다.
이번 주차 과제하시느라 수고 많으셨습니다🫶

Comment on lines +55 to +56
amazonS3.putObject(bucketName, file.getOriginalFilename(), file.getInputStream(), null);
amazonS3.setObjectAcl(bucketName,file.getOriginalFilename(), CannedAccessControlList.PublicRead);
Copy link
Member

Choose a reason for hiding this comment

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

PublicRead로 설정해주신 점 좋아요!👍

Comment on lines +1 to +5
1. IAM에서 AmazonS3FullAccess 권한을 가진 사용자 생성 -> application.yml의 accessKey와 secretKey에 입력<br>
2. ```getUrl```로 객체 URL 가져온 후, List<String>에 넣어 index.html에 반환하기<br>
a. ```listObjects```메서드로 버킷의 ```ObjectListing```객체에 대한 정보를 제공하는 객체를 반환<br>
b. ```getObjectSummaries``` 메서드를 사용하여 각 객체가 버킷의 단일 ```ObjectSummary``` 객체를 나타내는 S3 객체 목록을 가져옴<br>
c . 반복문을 돌며 summary에서 key 가져와 url을 list에 저장<br>
Copy link
Member

Choose a reason for hiding this comment

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

AmazonS3FullAccess 권한을 가진 사용자를 따로 만들어서 활용해주셨네요!
실습에서는 간편하게 하기 위해 루트 사용자의 엑세스 키를 활용해주었는데, 이 방법이 훨씬 안전하고 좋아요👍👍

@seojeewon seojeewon merged commit ebccc3c into main Nov 18, 2023
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