Skip to content

fix : 이미지 권한 수정 - #149

Merged
MINO1020 merged 1 commit into
developfrom
148-fix-이미지-s3-권한-수정
Mar 26, 2026

Hidden character warning

The head ref may contain hidden characters: "148-fix-\uc774\ubbf8\uc9c0-s3-\uad8c\ud55c-\uc218\uc815"
Merged

fix : 이미지 권한 수정#149
MINO1020 merged 1 commit into
developfrom
148-fix-이미지-s3-권한-수정

Conversation

@MINO1020

@MINO1020 MINO1020 commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #148

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • s3이미지 권한 수정
  • 작업 내용 2

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 이미지 파일 저장소의 접근 권한 처리가 개선되었습니다.

@MINO1020 MINO1020 self-assigned this Mar 26, 2026
@MINO1020 MINO1020 linked an issue Mar 26, 2026 that may be closed by this pull request
@MINO1020
MINO1020 merged commit 3f6c677 into develop Mar 26, 2026
1 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Mar 26, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a0d1c9cb-2781-41f1-9e16-410cd8a6cac4

📥 Commits

Reviewing files that changed from the base of the PR and between 75ef3c0 and b8ffe49.

📒 Files selected for processing (1)
  • src/main/java/teamficial/teamficial_be/domain/profile/service/PreSignedUrlService.java

Walkthrough

S3 객체 이동 작업에서 CopyObjectRequest를 사용하여 복사된 객체에 PublicRead 접근 제어 목록을 명시적으로 설정합니다. 기존의 간단한 copyObject 호출을 대신하여 더 세밀한 권한 관리를 구현합니다.

Changes

Cohort / File(s) Summary
S3 이미지 권한 설정
src/main/java/teamficial/teamficial_be/domain/profile/service/PreSignedUrlService.java
moveTempImageToPost 메서드에서 CopyObjectRequest를 사용하여 S3 객체 복사 시 CannedAccessControlList.PublicRead 권한을 명시적으로 설정합니다. CopyObjectRequestObjectMetadata 임포트 추가됨.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

🐛 fix

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 148-fix-이미지-s3-권한-수정

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

[fix] 이미지 s3 권한 수정

1 participant